Skip to content
Snippets Groups Projects
Commit e8e1d7e6 authored by Vadim ProductEngine's avatar Vadim ProductEngine
Browse files

STORM-318 FIXED The Advanced menu shortcut was broken under Linux.

Reason: The old shortcut (Ctrl+Alt+D) was eaten by some window managers.

Changes:
- Changed the shortcut to Ctrl+Alt+Shift+D.
- Moved the appropriate menu item from "Advanced > Shortcuts" to "World > Show"
  (so that it's not in the menu it triggers) and made it visible.
  The old shortcut is still available but marked as legacy.

Submitting on behalf of Boroondas Gupte.
parent a9eb6395
No related branches found
No related tags found
No related merge requests found
...@@ -177,6 +177,7 @@ Boroondas Gupte ...@@ -177,6 +177,7 @@ Boroondas Gupte
SNOW-610 SNOW-610
SNOW-624 SNOW-624
SNOW-737 SNOW-737
STORM-318
VWR-233 VWR-233
VWR-20583 VWR-20583
VWR-20891 VWR-20891
......
...@@ -359,6 +359,18 @@ ...@@ -359,6 +359,18 @@
<menu_item_check.on_check <menu_item_check.on_check
control="NavBarShowParcelProperties" /> control="NavBarShowParcelProperties" />
</menu_item_check> </menu_item_check>
<menu_item_separator />
<menu_item_check
label="Advanced Menu"
name="Show Advanced Menu"
shortcut="control|alt|shift|D">
<on_check
function="CheckControl"
parameter="UseDebugMenus" />
<on_click
function="ToggleControl"
parameter="UseDebugMenus" />
</menu_item_check>
</menu> </menu>
<menu_item_separator/> <menu_item_separator/>
...@@ -1526,6 +1538,18 @@ ...@@ -1526,6 +1538,18 @@
<menu_item_call.on_click <menu_item_call.on_click
function="View.DefaultUISize" /> function="View.DefaultUISize" />
</menu_item_call> </menu_item_call>
<!-- This second, alternative shortcut for Show Advanced Menu is for backward compatibility. The main shortcut has been changed so it's Linux-friendly, where the old shortcut is typically eaten by the window manager. -->
<menu_item_check
label="Show Advanced Menu - legacy shortcut"
name="Show Advanced Menu - legacy shortcut"
shortcut="control|alt|D">
<on_check
function="CheckControl"
parameter="UseDebugMenus" />
<on_click
function="ToggleControl"
parameter="UseDebugMenus" />
</menu_item_check>
<menu_item_separator/> <menu_item_separator/>
...@@ -1693,23 +1717,6 @@ ...@@ -1693,23 +1717,6 @@
<menu_item_call.on_click <menu_item_call.on_click
function="View.ZoomOut" /> function="View.ZoomOut" />
</menu_item_call> </menu_item_call>
<menu_item_separator
visible="false"/>
<!-- Made invisible to avoid a dissonance: menu item toggles the menu where it is located. EXT-8069.
Can't be removed, to keep shortcut workable.
-->
<menu_item_check
label="Show Advanced Menu"
name="Show Advanced Menu"
shortcut="control|alt|D"
visible="false">
<on_check
function="CheckControl"
parameter="UseDebugMenus" />
<on_click
function="ToggleControl"
parameter="UseDebugMenus" />
</menu_item_check>
</menu> <!--Shortcuts--> </menu> <!--Shortcuts-->
<menu_item_separator/> <menu_item_separator/>
...@@ -1732,7 +1739,6 @@ ...@@ -1732,7 +1739,6 @@
function="ToggleControl" function="ToggleControl"
parameter="QAMode" /> parameter="QAMode" />
</menu_item_check> </menu_item_check>
</menu> </menu>
<menu <menu
create_jump_keys="true" create_jump_keys="true"
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment