Merge lp:~ivanfateev/pantheon-files/fix-1380139 into lp:~elementary-apps/pantheon-files/trunk
Status: | Rejected | ||||
---|---|---|---|---|---|
Rejected by: | Jeremy Wootten | ||||
Proposed branch: | lp:~ivanfateev/pantheon-files/fix-1380139 | ||||
Merge into: | lp:~elementary-apps/pantheon-files/trunk | ||||
Diff against target: |
153 lines (+70/-20) 2 files modified
src/CMakeLists.txt (+2/-0) src/View/AbstractDirectoryView.vala (+68/-20) |
||||
To merge this branch: | bzr merge lp:~ivanfateev/pantheon-files/fix-1380139 | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Jeremy Wootten | Disapprove | ||
Review via email: mp+257797@code.launchpad.net |
Description of the change
Added support for gnome-sushi previewer according to this bug
https:/
How to test:
- install gnome-sushi. Select file, hit space, previewer should open
- remove gnome-sushi. Select file, hit space, pantheon-files should behave as was designed before, open select file
Unmerged revisions
- 1815. By Ivan Fateev
-
removed debug messages about previewer
- 1814. By Ivan Fateev
-
- added full support for previewer gnome-sushi
- added enum PreviewerType - maybe files will support more previewers
- if gnome-sushi is not installed - behaviour is default (open selected file) - 1813. By Ivan Fateev
-
integrated gnome-sushi previewer via dbus. TODO: check whether gnome-sushi is installed. Added gdk-x11-3.0 as dependency
The problem with this is that installing gnome-sushi also installs Nautilus, which does not seem desirable. Is it possible to remove this dependency on Nautilus?