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 699290 - API request: MenuItem has no function to set the accelerator key
API request: MenuItem has no function to set the accelerator key
Status: RESOLVED NOTABUG
Product: gtk+
Classification: Platform
Component: Widget: GtkMenu
unspecified
Other Linux
: Normal enhancement
: ---
Assigned To: gtk-bugs
gtk-bugs
Depends on:
Blocks:
 
 
Reported: 2013-04-30 08:23 UTC by Luc Pi
Modified: 2013-05-11 08:23 UTC
See Also:
GNOME target: ---
GNOME version: ---


Description Luc Pi 2013-04-30 08:23:01 UTC 
There is no direct function to set the accelerator on a MenuItem.

Only by reading GtkMenuItem source code, I could find out that its Label is actually an AccelLabel, and that we can set its accelerator key. Then we need to know that the MenuItem is also a Bin, and that its child is the AccelLabel. Then we can call gtk_accel_label_set_accel().


I suggest to add:

void gtk_menu_item_set_accel (GtkMenuItem    *menu_item,
                              guint           accelerator_key,
                              GdkModifierType accelerator_mods);



(I can provide a patch, if it makes sense)
Comment 1 Matthias Clasen 2013-05-04 23:53:46 UTC 
Doesn't make much sense, no. There's nothing wrong with directly accessing the accel label inside the menuitem, if you need to.
Comment 2 Luc Pi 2013-05-05 03:42:12 UTC 
(similar to bug 699289, extract from bug 699289, comment 2)

From an application developer point of view, looking at the documentation, it is hidden. You have to discover it. You need a certain knowledge of GTK+ to figure it out. Certainly a piece of cake for an advanced GTK+ developer, but not for everybody. Maybe the API could be a bit more consistent here.


I think that this is at least missing in the documentation.
Comment 3 Matthias Clasen 2013-05-06 11:13:52 UTC 
> You need a certain knowledge of GTK+ to figure it out.

Thats generally true when using any complex library.

Closing this since I don't want to carry around more open documentation bugs -
feel free to reopen if you have a patch to attach.
Comment 4 Luc Pi 2013-05-11 08:23:45 UTC 
fixed by bug 699289