[Webkit-unassigned] [Bug 90437] [GTK] Don't show accel labels in context menu items

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=90437





--- Comment #3 from Carlos Garcia Campos <cgarcia at igalia.com>  2012-07-03 08:31:49 PST ---
(In reply to comment #2)
> (From update of attachment 150554 [details])
> I'm not totally sure I understand the patch. Is the issue that accelerator labels are still showing up or that they are both showing up and active? Does gtk_accel_label_set_accel_closure have the side effect of hiding accelerator labels and disabling the accelerator?

This doesn't affect the action accelerator which is not active while the context menu is shown. For example, if you create a GtkAction for Paste with CTRL+V as shortcut, and _Paste as label. When the context menu is not active you can do CTRL+V to trigger the paste action. When the context menu is active you can just press 'p' to trigger the action, but CTRL+V won't work. When the context menu is built, GTK+ doesn't know whether the menu is part of a menubar or a popup, and the shortcut (CTRL + V) is shown at the right of the menu item. To avoid it, we can simply reset the accel closure of the label contained in the menu item (in case it's an accel label, of course).

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.