Thanks Simon, attached is a patch with the comments you did. > The patch does double the "No gtk.AccelGroup in the top level window" warnings printed in the logs. > I don't think those are really useful for anything I would remove that (in a separate patch for master) and if at all add a comment there. May be we can remove the callback when the accelerator is set, and add it again if the accelerator is changed. I did not wanted to do it in this patch because the patch would be more difficult to test/understand/review/accept but can be a god improvement. Gonzalo <div class="gmail_quote"><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;"> Regards, Simon _______________________________________________ Sugar-devel mailing list <a href="mailto:Sugar-devel@lists.sugarlabs.org" target="_blank">Sugar-devel@lists.sugarlabs.org</a> <a href="http://lists.sugarlabs.org/listinfo/sugar-devel" target="_blank">http://lists.sugarlabs.org/listinfo/sugar-devel</a> </blockquote></div>