[Geany-devel] Geany.txt cleanup

[Lex Trotman]

Hi Nick,

Sorry I missed your reply.

There are two problems, one is solvable, the other may be terminal for now.

Solvable, lots of the menu items have NULL widget pointers in
kb->menu_item so widget_accelerator_remove and ..._add are not called
in keybindings_update_combo.

The other I'm not sure at this point how to solve :-(

For stock items Glade sets the accelerator when creating the menu_item
using an accelerator map that you can't get hold of, so you can't
delete the setting.  There doesn't seem to be any way of telling it to
not set the accel for a stock item.  I'm not sure what happens if you
use the same stock for more than one menu item??  Just one more item
on my hate list for Glade :-)

Of course getting rid of glade altogether is a solution since
gtk_image_menu_item_new_from_stock can be passed an accel group so if
we called it instead of glade we would know what was used, but lots of
work required for probably not much gain :-(

Cheers
Lex


2009/10/2 Nick Treleaven <nick.treleaven at btinternet.com>:
> On Wed, 30 Sep 2009 10:37:54 +1000
> Lex Trotman <elextr at gmail.com> wrote:
>
>> > But at least I was partially right: the displayed accelerators on the
>> > stock menu items are not updated, not even after a restart.
>> >
>>
>> Nick and I had a long discussion about accels not changing in the past
>> when I discovered the <super> problem.  I thought that problem had
>> been fixed as part of making accels change when keybindings are
>> changed, without reload being needed.  It is possible to change the
>> accel labels even on stock items, you just have to remove the
>> accelerator before adding a new one, see attached <super> problem demo
>
> I tried to do this for stock items also but it didn't seem to work, I
> sent a reply to the list:
> http://lists.uvena.de/geany-devel/2009-August/001119.html
>
> Regards,
> Nick
> _______________________________________________
> Geany-devel mailing list
> Geany-devel at uvena.de
> http://lists.uvena.de/cgi-bin/mailman/listinfo/geany-devel
>