Bug 384861

Summary: Auto-hide panel should raise when global menu is triggered via Alt-key
Product: [Plasma] plasmashell Reporter: Antonio Russo <aerusso>
Component: Global Menu widgetAssignee: Kai Uwe Broulik <kde>
Status: RESOLVED FIXED    
Severity: normal CC: kde, plasma-bugs-null
Priority: NOR    
Version First Reported In: 5.10.5   
Target Milestone: 1.0   
Platform: Debian unstable   
OS: Linux   
Latest Commit: Version Fixed/Implemented In:
Sentry Crash Report:

Description Antonio Russo 2017-09-19 18:48:14 UTC
As described in https://bugs.kde.org/show_bug.cgi?id=376726 , the global menu is now properly triggered when a key (say alt-F for the File menu). However, if the panel containing the global menu is set to "auto-hide", the panel itself may not be visible.

IMO, the panel should un-hide when its contained global menu is activated, the same way that activating the application menu (launcher or K menu) raises the containing panel.
Comment 1 Kai Uwe Broulik 2017-09-19 18:50:40 UTC
Agreed.

David, why doesn't the "if transient is shown" hack you made for panels work for global menu? Otherwise I'll just fix it to become requesting input whilst a menu is open.
Comment 2 David Edmundson 2017-09-20 09:19:42 UTC
Two reasons:

1) Because the QMenus don't have a transient parent (and thus panel has no children).

2) My "hack" / "awesome fix" never causes activation only prevents closing.


I'll fix this.
Comment 3 David Edmundson 2017-09-20 23:04:42 UTC
Git commit 76b276056e0cb39423cccfd2e429a2d65dea3c06 by David Edmundson.
Committed on 20/09/2017 at 23:04.
Pushed by davidedmundson into branch 'Plasma/5.11'.

Unhide autohidden panel when using global menu

Summary:
This sets a transient parent on the shown menus, meaning a panel knows
to stay open whilst it has any menu showing.

We also pulse setStatus() to trigger activating an autohidden panel if
activated via a shortcut.

Test Plan:
Ran menu
Using both mouse and keyboard shortcuts navigated a menu
and submenus

Reviewers: #plasma, broulik

Reviewed By: #plasma, broulik

Subscribers: broulik, plasma-devel

Tags: #plasma

Differential Revision: https://phabricator.kde.org/D7883

M  +8    -0    applets/appmenu/lib/appmenuapplet.cpp

https://commits.kde.org/plasma-workspace/76b276056e0cb39423cccfd2e429a2d65dea3c06