[Okular-devel] Review Request: Fix for the bug-166645

[Albert Astals Cid]

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


I just realized we are not doing it right, the close action should be in of the shell, not of the part (as the open action is in the shell and not in the part).

Can you have a look for that?  This way you won't need the part.rc modification.

The only issue i can see is knowing the file was closed internally (by those of one links we talked about earlier) but i guess you can always add a signal to the part and connect from the shell.

- Albert Astals Cid


On Aug. 15, 2012, 10:05 p.m., Jaydeep Solanki wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/105965/
> -----------------------------------------------------------
> 
> (Updated Aug. 15, 2012, 10:05 p.m.)
> 
> 
> Review request for Okular.
> 
> 
> Description
> -------
> 
> This patch is a fix for the bug-166645, as listed here (https://bugs.kde.org/show_bug.cgi?id=166645)
> 
> Opens a new instance of Okular for every other file that is opened.
> 
> 
> Diffs
> -----
> 
>   part.h d9a5cc4 
>   part.cpp 7c1ca71 
>   part.rc 33d3829 
>   shell/shell.cpp 8b42a27 
>   shell/shell.rc 93fbc41 
> 
> Diff: http://git.reviewboard.kde.org/r/105965/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Jaydeep Solanki
> 
>

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