Review Request 113175: Always use an external viewer application to view files
Sven Brauch
svenbrauch at googlemail.com
Tue Oct 8 16:13:53 BST 2013
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/113175/
-----------------------------------------------------------
(Updated Oct. 8, 2013, 3:13 p.m.)
Review request for KDE Base Apps.
Repository: ark
Description
-------
This patch makes ark always use an external viewer application instead of using the clunky internal preview thing. The internal viewer would just embed a plain kpart into a window, but without providing any of the XMLGUI or whatever from that part. Thus, when you for example clicked a PDF, you couldn't print it. The advantages of the internal viewer are imo overall quite questionable, and are far outweighted by its disadvantages.
Plus, it removes code ;)
Diffs
-----
part/arkviewer.cpp 053cd1c0502d3bb88895dc8d3653eaea9e6c3c83
Diff: http://git.reviewboard.kde.org/r/113175/diff/
Testing
-------
Clicking files opens them in the default application, as it should.
Thanks,
Sven Brauch
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20131008/7ffa8b4b/attachment.htm>
More information about the kde-core-devel
mailing list