> Does the following patch give good results? > > Index: src/xmenu.c > =================================================================== > RCS file: /cvsroot/emacs/emacs/src/xmenu.c,v > retrieving revision 1.295 > diff -u -r1.295 xmenu.c > --- src/xmenu.c 7 Aug 2005 12:33:19 -0000 1.295 > +++ src/xmenu.c 9 Nov 2005 19:08:26 -0000 > @@ -3498,6 +3498,7 @@ > /* Help display under X won't work because XMenuActivate contains > a loop that doesn't give Emacs a chance to process it. */ > menu_help_frame = f; > + XSETFRAME (Vmenu_updating_frame, f);
It doesn't work. Vmenu_updating_frame is nil initially on a new frame, because this change is in `xmenu_show', and in the non-toolkit version `xmenu_show' is called only after clicking on the menu bar. So initially some icons in the toolbar are disabled. After the first mouse click on the menu bar, Vmenu_updating_frame gets a non-nil value, so icons on the toolbar become enabled. But in any case, menu items in sub-menus on the menu bar are still disabled regardless of the value of Vmenu_updating_frame. Maybe setting of Vmenu_updating_frame should be done in Fx_create_frame for the non-toolkit version. But I still doubt that setting of Vmenu_updating_frame is necessary. My previous change (i.e. using `(or menu-updating-frame (selected-frame))') assumes that if `menu-updating-frame' is nil, this means that the menu frame is the same as the selected frame. And this would work for the non-toolkit version. -- Juri Linkov http://www.jurta.org/emacs/ _______________________________________________ Emacs-pretest-bug mailing list [email protected] http://lists.gnu.org/mailman/listinfo/emacs-pretest-bug
