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 113112 - Cannot determine whether a widget has an attached menu
Cannot determine whether a widget has an attached menu
Status: RESOLVED FIXED
Product: gtk+
Classification: Platform
Component: Widget: GtkMenu
2.2.x
Other Solaris
: Normal normal
: ---
Assigned To: gtk-bugs
gtk-bugs
AP3
Depends on:
Blocks:
 
 
Reported: 2003-05-16 12:13 UTC by padraig.obriain
Modified: 2011-02-04 16:17 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Proposed patch (3.19 KB, patch)
2004-04-07 14:25 UTC, padraig.obriain
none Details | Review
Corrected patch (3.19 KB, patch)
2004-04-29 10:22 UTC, padraig.obriain
accepted-commit_now Details | Review

Description padraig.obriain 2003-05-16 12:13:18 UTC
I can call gtk_menu_attach_to_widget to attach a menu to a widget but given
a widget it is not possible to determine whether it has an attached menu.

Would it be posible to add a statement like 

g_object (G_OBJECT (attach_widget), "gtk-attached-menu", menu); 

to the implementation of gtl_menu_attach_to_widget?
Comment 1 Owen Taylor 2003-05-19 23:28:20 UTC
Well, we'd certainly want to encapsulate that in

 gtk_menu_get_for_attach_widget()

or something, rather than making the gobject key part
of the API. 

But other than that, sounds reasonable.
Comment 2 Owen Taylor 2003-05-19 23:51:48 UTC
Oh, one thing that woudl have to be dealt with  ... there can be
multiple menus attached to a single widget.
Comment 3 Calum Benson 2003-06-25 22:50:55 UTC
Marking AP2 to reflect a11y team's assessment of a11y impact.
Comment 4 Calum Benson 2003-08-07 16:15:41 UTC
Apologies for spam... marking as GNOMEVER2.3 so it appears on the official GNOME
bug list :)
Comment 5 Owen Taylor 2004-02-29 16:41:40 UTC
No patch, no final API proposal
Comment 6 padraig.obriain 2004-04-07 14:25:32 UTC
Created attachment 26434 [details] [review]
Proposed patch
Comment 7 padraig.obriain 2004-04-29 10:22:29 UTC
Created attachment 27199 [details] [review]
Corrected patch

Corrects problem with previous patch when gtk_menu_detach is called.
Comment 8 Matthias Clasen 2004-05-06 03:00:40 UTC
Comment on attachment 27199 [details] [review]
Corrected patch

Looks good to me. You may want to add a note regarding memory management to the
docs: The list is owned by GTK+ and must not be modified.
Comment 9 padraig.obriain 2004-05-06 07:35:51 UTC
Patch committed to CVS HEAD.