Bug 323267

Summary: Adding and removing bookmarks in the thumbnail panel acts on the current page instead
Product: [Applications] okular Reporter: Jaan Vajakas <jaanvajakas>
Component: generalAssignee: Okular developers <okular-devel>
Status: RESOLVED FIXED    
Severity: normal CC: aacid, mailson
Priority: NOR    
Version: 0.17.60   
Target Milestone: ---   
Platform: Compiled Sources   
OS: Linux   
Latest Commit: http://commits.kde.org/okular/b43a4337af47d1b1871dd5a8ff1a62de972cc167 Version Fixed In: 4.11.5

Description Jaan Vajakas 2013-08-07 19:25:10 UTC 
Although the pop-up menu appearing at right click in Thumbnails view correctly displays the number of the page whose thumbnail is under the mouse cursor, the "Add bookmark" and "Remove bookmark" commands in the menu act on the current page, not to the page under the mouse cursor.

Reproducible: Always

Steps to Reproduce:
1. Open a multi-page document. Scroll to the beginning of page 1 if you aren't there already.
2. Open the Thumbnails view in the Navigation panel.
3. Right-click on the thumbnail of page 2; click "Add bookmark".
4. Right-click on the thumbnail of page 2; click "Remove bookmark".
Actual Results:  
"Add bookmark" adds a bookmark to the page 1. "Remove bookmark" removes this bookmark.

Expected Results:  
"Add bookmark" should add a bookmark to the page 2. "Remove bookmark" should remove a bookmark from page 2.
Comment 1 Albert Astals Cid 2013-08-17 14:56:40 UTC 
Mailson you broke this.
Comment 2 Albert Astals Cid 2013-10-12 18:31:22 UTC 
Mailson, you alive?
Comment 3 Mailson Menezes 2013-10-14 11:11:05 UTC 
Wow. Sorry. I'll take a look on that.
Comment 4 Pinak Ahuja 2013-10-14 12:39:44 UTC 
You're laptop came with windows 8?  Check out this link:
https://help.ubuntu.com/community/UEFI
Comment 5 Pinak Ahuja 2013-10-14 12:40:01 UTC 
Sorry wrong list
Comment 6 Albert Astals Cid 2013-12-16 22:35:20 UTC 
Mailson?
Comment 7 Albert Astals Cid 2013-12-29 18:10:31 UTC 
Git commit b43a4337af47d1b1871dd5a8ff1a62de972cc167 by Albert Astals Cid, on behalf of Mailson Menezes.
Committed on 29/12/2013 at 18:09.
Pushed by aacid into branch 'KDE/4.11'.

Fix adding and removing bookmarks in the thumbnail panel

Fix adding and removing bookmarks in the thumbnail panel

Add and removal of bookmarks via the thumbnail context menu was acting
on the current viewport instead of the clicked page. This patch checks
whether it should act on the viewport or in the right clicked page.
FIXED-IN: 4.11.5

M  +22   -4    part.cpp

http://commits.kde.org/okular/b43a4337af47d1b1871dd5a8ff1a62de972cc167