[Okular-devel] Review Request 127351: Added ability to view embedded files

Daniel Lichtenberger daniel.lichtenberger at gmail.com
Wed Mar 16 16:59:37 UTC 2016


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

(Updated March 16, 2016, 4:59 p.m.)


Review request for Okular.


Changes
-------

Made indentation consistent (within methods).


Bugs: 295567
    http://bugs.kde.org/show_bug.cgi?id=295567


Repository: okular


Description
-------

The selected embedded file(s) are extracted to QDir::tempPath() and the
default application associated with the file type is launched with KRun.

Double-click on an embedded file also launches the view action.

The temporary files are cleaned up automatically when the dialog is
closed.


Diffs (updated)
-----

  ui/embeddedfilesdialog.h f3e2d038f4ffe2b64af15c94c7ec0cfb8a68834c 
  ui/embeddedfilesdialog.cpp b400e879b2849a931e069669c984a6b578ffd536 
  ui/guiutils.h fffe04488b27d92d9a7bc623da653d0c1df87ae8 
  ui/guiutils.cpp 3bf3ad64b57396792c3aa33298265561ee35ed1a 

Diff: https://git.reviewboard.kde.org/r/127351/diff/


Testing
-------

Tested with 

http://www.opf-labs.org/format-corpus/pdfCabinetOfHorrors/fileAttachment.pdf 

and the PDF file in

http://www.sdbtransfer.de/app/download/10536876499/SuperSauber1_EDASXbau.zip?t=1453751464


Thanks,

Daniel Lichtenberger

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/okular-devel/attachments/20160316/f5e890d2/attachment.html>


More information about the Okular-devel mailing list