Inkscape

Merge lp:~h-h-h/inkscape/context-menu-key-enabling into lp:~inkscape.dev/inkscape/trunk

  1. context-menu-key-enabling
  2. Merge into trunk
Proposed by h-h-h
Status: Merged
Approved by: Mc
Approved revision: 14972
Merged at revision: 14973
Proposed branch: lp:~h-h-h/inkscape/context-menu-key-enabling
Merge into: lp:~inkscape.dev/inkscape/trunk
Diff against target: 15 lines (+5/-0)
1 file modified
src/ui/tools/tool-base.cpp (+5/-0)
To merge this branch: bzr merge lp:~h-h-h/inkscape/context-menu-key-enabling
Reviewer Review Type Date Requested Status
Mc Approve
Review via email: mp+297114@code.launchpad.net

Description of the change

Enabled using the context menu key additionally to the already working Shift+F10 key combination to show the context menu ("tool-base.cpp")

To post a comment you must log in.
Revision history for this message
Mc (mc...) wrote :

Nice idea, thanks ! (merged)

review: Approve
Revision history for this message
jazzynico (jazzynico) wrote :

Oh, we need to update the keys and mouse reference again...
Nice idea anyway.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/ui/tools/tool-base.cpp'
2--- src/ui/tools/tool-base.cpp 2015-12-06 12:53:50 +0000
3+++ src/ui/tools/tool-base.cpp 2016-06-10 21:27:33 +0000
4@@ -683,6 +683,11 @@
5 }
6 break;
7
8+ case GDK_KEY_Menu:
9+ sp_event_root_menu_popup(desktop, NULL, event);
10+ ret = TRUE;
11+ break;
12+
13 case GDK_KEY_F10:
14 if (MOD__SHIFT_ONLY(event)) {
15 sp_event_root_menu_popup(desktop, NULL, event);