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

[Albert Astals Cid]

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



Why did you develop this against the framework branch? You don't want this to end up in a release people will actually use?

I was going to commit (with some spacing fixes) to Applications/16.04 but your patch doens't apply.

- Albert Astals Cid


On March 16, 2016, 4:59 p.m., Daniel Lichtenberger wrote:
> 
> -----------------------------------------------------------
> 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.
> 
> 
> 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
> -----
> 
>   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/20160321/2c258106/attachment.html>