Review Request 113175: Always use an external viewer application to view files

Sven Brauch svenbrauch at googlemail.com
Tue Oct 8 16:17:17 BST 2013


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/113175/
-----------------------------------------------------------

(Updated Oct. 8, 2013, 3:17 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 (updated)
-----

  part/arkviewer.h bb41472eaec985e2e1b3d9c2f7c257c949316bf4 
  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/b86bba7f/attachment.htm>


More information about the kde-core-devel mailing list