----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/116700/#review55311 -----------------------------------------------------------
This review has been submitted with commit 89fd56c23bc19812d3514991a6eeb9f27c2cb742 by Albert Astals Cid on behalf of Jonathan Doman to branch KDE/4.13. - Commit Hook On March 20, 2014, 7:33 p.m., Jonathan Doman wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/116700/ > ----------------------------------------------------------- > > (Updated March 20, 2014, 7:33 p.m.) > > > Review request for Okular. > > > Bugs: 331872 and 332238 > http://bugs.kde.org/show_bug.cgi?id=331872 > http://bugs.kde.org/show_bug.cgi?id=332238 > > > Repository: okular > > > Description > ------- > > Currently, tabs can only be created by opening files through okular's file > dialog. This patch allows external sources (e.g. file browsers) to open > documents in tabs of existing windows. > > Basically, whenever okular is launched it uses D-Bus to search for existing > okular instances that are capable of handling its documents. An eligible > window will have enough space for all the documents (space depends on whether > tabs are allowed or not), and be located on the current desktop. If there are > multiple eligible windows, one is chosen "at random" (the first one in the > D-Bus listing). This metric can be changed in the future, but is the simplest > for now. > > To facilitate the D-Bus interaction, two methods were added: "canOpenDocs" is > used to find an eligible window, "openDocument" is used to actually open a > single document. Additionally, "tryRaise" was modified to work on non-unique > windows. > > I previously thought that okular should behave similar to a web browser, but > now I think they are very different. Most people open a web browser and then > search and open links completely within it. However, most people will use > okular by clicking on files in a file browser. Therefore, I don't think > okular should ever open tabs in a window on a different desktop. > > One shortcoming occurs when one wants to open a document in its own window, > but instead gets put into a tab. There is no way to "detach" a tab, so it's > necessary to then open an empty okular instance and use the file dialog. > Detaching tabs should be possible, but I guess there are concerns about how > to do so efficiently. > > > Diffs > ----- > > shell/shell.cpp 01a9f1b1dc1992a4aa1c87330940b27b727608e5 > shell/shell.h 3b9aae061cd6be0c7a86c885fc06d00d9275bd50 > part.h 010e9de1f2b5c27531a48b943d821ccc3f3f7205 > part.cpp 4ce7e28e1071772921e6292e61a88c905a62d7f6 > shell/main.cpp e0ca587ba167c4020d5af5bd33a2dc1b4923cac4 > ui/thumbnaillist.h 61601c228512772fd46abc27468523aef562c3fa > ui/sidebar.cpp 2474db8c357f6bfd1a9b6bd75091f2eb7b7b7693 > ui/pageview.cpp dd4199450672c18ebfa146327d8e9b7eb034ddc8 > ui/sidebar.h 036d7788096366d6bab7d7f2a41d55b7a31d303a > ui/pageview.h 577b908633bd0778df33d0a15961ab16071a1500 > ui/thumbnaillist.cpp 72b557e6624e8229cf1e5e4c5dc69ed77fec54cb > > Diff: https://git.reviewboard.kde.org/r/116700/diff/ > > > Testing > ------- > > > Thanks, > > Jonathan Doman > >
_______________________________________________ Okular-devel mailing list [email protected] https://mail.kde.org/mailman/listinfo/okular-devel
