After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 699678 - Child menus are broken
Child menus are broken
Status: RESOLVED FIXED
Product: gnome-shell
Classification: Core
Component: general
unspecified
Other Linux
: Normal normal
: ---
Assigned To: gnome-shell-maint
gnome-shell-maint
: 700528 700531 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2013-05-04 18:19 UTC by Giovanni Campagna
Modified: 2013-05-17 14:44 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
Screenshot with multiple menus open (732.01 KB, image/png)
2013-05-04 18:19 UTC, Giovanni Campagna 		  Details
PopupMenuManager: Fix child menus (1.27 KB, patch)
2013-05-14 16:58 UTC, Florian Müllner 		committed Details | Review

Description Giovanni Campagna 2013-05-04 18:19:07 UTC 
Created attachment 243301 [details]
Screenshot with multiple menus open

Open the user menu, click on the chat status changer. Nothing opens, but the PopupMenuManager gets confused, so you can open all menus at once.

Wonderful screenshot attached.
Comment 1 Florian Müllner 2013-05-14 16:58:42 UTC 
Created attachment 244207 [details] [review]
PopupMenuManager: Fix child menus

Since commit c84dc6254d2d65c, popup menus are closed automatically
when another menu opens (to catch the case where a menu is opened
by keyboard shortcut, which wasn't handled before). However in the
case of child menus, both child and parent are expected to be visible,
so handle this case explicitly.
Comment 2 Jasper St. Pierre (not reading bugmail) 2013-05-14 17:07:28 UTC 
Review of attachment 244207 [details] [review]:

OK.
Comment 3 Florian Müllner 2013-05-14 17:08:52 UTC 
Attachment 244207 [details] pushed as b588ae4 - PopupMenuManager: Fix child menus
Comment 4 Jasper St. Pierre (not reading bugmail) 2013-05-17 14:27:57 UTC 
*** Bug 700528 has been marked as a duplicate of this bug. ***
Comment 5 Florian Müllner 2013-05-17 14:44:55 UTC 
*** Bug 700531 has been marked as a duplicate of this bug. ***