Re: [PATCH] Gtk::MenuItem::set_accel_path()

I have committed your patch to git's gtkmm-3-22 branch (gtkmm3) and master branch (gtkmm4). Usually a bug report in Bugzilla is a better place than gtkmm-list for attaching patches.


On 2018-02-08 21:26, Christian Schoenebeck wrote:

I just investigated a bug in gtkmm3 (which did not exist with gtkmm2 before)
which happens when using Gtk::MenuItem::set_accel_path() to assign a global
keyboard accelerator to a menu item. Since gtkmm3 the respective keyboard
accelerator action is still triggered, but the accelerator keys were no longer
displayed in the menu.

I first thought this to be a bug in Gtk(3), so I started the discussion about
this issue there:

But meanwhile it rather looks like a bug in gtkmm3 to me. Attached you find a
quick hack against gtkmm3 which resolved this bug for me. Like I mentioned in
the last post on the gtk list, I have no idea why that acceleration label was
created explicitly in the gtkmm "MenuItem" class there. Was it still there for
purpose or was it just some legacy code?

Note that (in gtk3 at least) a GtkAccelerationLabel is automatically always
created as child widget by GtkMenuItem even if no acceleration keys are used
at all for the menu item.


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]