Object context menu (right-click) acts on bottom-most instead of top-most object under cursor position

[su_v]

Regression in current trunk:
Object context menu (right-click) acts on bottom-most instead of top-most object under cursor position

Steps to reproduce:
1) open attached sample file
2) right-click on each of the text objects and compare the context menu entries

Expected result:
Same context menu with text-specific entries for each of the text objects (top-most object at current position)

Actual result:
Context menu specific to bottom-most (unlocked) object is opened on RMB click

Workaround:
1) select text object (LBM click)
2) open context menu with 'Shift+F10' (without RMB click)

On Mac OS X 10.5.8. (32bit) and OS X 10.7.2 (64bit):
- not reproduced with Inkscape 0.46, 0.47, 0.48.x and 0.48+devel <= r10577
- reproduced with Inkscape 0.48+devel >= r10581
The regression was possibly introduced with the merge of the rendering cache branch in revision 10579.

This is a follow-up report to:
Bug #962002 <image> context menu item 'Edit externally…' ignores current selection
The correct fix for bug #962002 is probably to make the right-click context menu work on the top-most selectable object under the current pointer position again.

(See also <http://article.gmane.org/gmane.comp.graphics.inkscape.devel/6897> - though I'm not sure how relevant this paper actually was for the implementation of the context menu)

[jazzynico (jazzynico)]

This one is very annoying to me...

[su_v (suv-lp)]

JazzyNico wrote:
> This one is very annoying to me...

Agreed. I stopped using the context menu in current trunk altogether (this is most unfortunate for images, because their specific context menu entries (image properties, edit externally) aren't available elsewhere nor as keyboard shortcuts).

IMHO this is a blocker for 0.49 (feel free to remove the tag again if you don't agree).

[John Smith (john-smithi)]

Patch attached that should fix this.
It uses the same logic as the left click to determine the top z-order item.

[John Smith (john-smithi)]

Fix committed as r11672.

[su_v (suv-lp)]

@John - many thanks for the fix! :)