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 630914 - PopupMenuManager does not connect to 'destroy' signal
PopupMenuManager does not connect to 'destroy' signal
Status: RESOLVED FIXED
Product: gnome-shell
Classification: Core
Component: general
unspecified
Other Linux
: Normal normal
: ---
Assigned To: Dan Winship
gnome-shell-maint
Depends on:
Blocks:
 
 
Reported: 2010-09-29 16:14 UTC by Giovanni Campagna
Modified: 2010-10-01 16:38 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
Connect PopupMenuManager to destroy signal (1.17 KB, patch)
2010-09-29 16:14 UTC, Giovanni Campagna 		committed Details | Review

Description Giovanni Campagna 2010-09-29 16:14:15 UTC 
When introducing support for dynamic menus in bug 622730 PopupMenuManager was meant to connect to the 'destroy' signal of PopupMenu, so that destroying a menu would be enough for having it removed.
Comment 1 Giovanni Campagna 2010-09-29 16:14:41 UTC 
Created attachment 171344 [details] [review]
Connect PopupMenuManager to destroy signal

PopupMenuManager now connects to the destroy signal of PopupMenu, so
that destroying a PopupMenu is now enough for removing it from the
relevant manager. Useful if menu come and go on-the-fly (like in
app views or sidebars).