Created attachment 2813 Fix menu/submenu creation and show Hey, Grégoire Gentil (which uses Xfce on the Touch Book) spotted a bug in libxfcegui4 (and maybe xfdesktop) about menu creation. He recently sent a mail on xfce-dev (but got no answer), and, after investigation, managed to fix that (for 4.4), the patch is here: According to http://library.gnome.org/devel/gtk-tutorial/stable/c1501.html there's no need to gtk_widget_show() the submenu after creating it, so it seems GTK+ gets confused in case it receives events one by one (like in the touchscreen). It seems that the correct behavior would be to: * submenu = gtk_menu_new(); * create and append the submenu items * create the menuitem * show the submenu items I'm not exactly sure what that means precisely for libxfcegui4, but in any case the gtk_widget_show(submenu) shouldn't be called after gtk_menu_new(). Attached patch should fix it, and I'm not sure about libxfce4ui.
(In reply to comment #0) > Created an attachment (id=2813) [details] > Fix menu/submenu creation and show > > Hey, > > Grégoire Gentil (which uses Xfce on the Touch Book) spotted a bug in > libxfcegui4 (and maybe xfdesktop) about menu creation. He recently sent a mail > on xfce-dev (but got no answer), and, after investigation, managed to fix that > (for 4.4), the patch is here: Here: http://git.alwaysinnovating.com/cgit.cgi/ai.openembedded.dev/commit/?id=c191416f78baf6e1aaf99d14915d1e8bec455ac5 The problem lies more in xfdesktop than libxfcegui4 in the end as it seems. > Attached patch should fix it, and I'm not sure about libxfce4ui. As the patch is against xfdesktop, there might not be anything related to libxfce4ui
Is it confirmed this patch works? It is indeed true you never call gtk_widget_show for menu's so it looks right to me.
(In reply to comment #2) > Is it confirmed this patch works? It is indeed true you never call > gtk_widget_show for menu's so it looks right to me. TBH, I didn't have a chance to test it yet, I just ported Grégoire's patch to 4.6 so it might be wise to test it first. I'll try to do that this evening, building on the Touch Book directly.
Fixed this in 84996fa.