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

[Albert Astals Cid]

> On Aug. 12, 2012, 11:17 p.m., Albert Astals Cid wrote:
> > "because there was no way to close the document"
> > 
> > That's totally incorrect, there's lots of ways of a document being closed. Have a look at all the possible calls to closeUrl in Part? How do you handle these?
> 
> Jaydeep Solanki wrote:
>     Not that, yes you can close the document programmatically, but that was not possible by the user. Like if I have one instance of Okular with Harry Potter opened, and if I want to open another book in the same instance, I can only do that after closing Harry Potter. So for the convenience of the user I have provided the "Close Document" action in File Menu, with that he can close the current document and open another document in the same instance.
>     
>     If I don't provide it, whenever the user try to open new document, he would have to "Quit" okular, and open it again, because till now there was no "close" available to user, there was only the "Quit" option.
> 
> Albert Astals Cid wrote:
>     You are not reading the code correctly. There are ways the user can close the document, e.g. if the document provides a close action link. How do you handle that?
> 
> Jaydeep Solanki wrote:
>     Sir, I didn't get you, can you please elaborate it a bit..

Don't call me Sir ;D I don't look at all like http://images.google.com/images?q=sir ;-)

What i mean is:
 * You have a document with a CloseAction
 * The user clicks on a button/link that activates the CloseAction
 * The document is closed


- Albert


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


On Aug. 12, 2012, 9:31 p.m., Jaydeep Solanki wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/105965/
> -----------------------------------------------------------
> 
> (Updated Aug. 12, 2012, 9:31 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.h 8430b9c 
>   shell/shell.cpp 8b42a27 
> 
> 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/20120813/7f7a2b08/attachment-0001.html>