changeset 37022:d7d30cecd891

(free_frame_menubar) [USE_MOTIF]: If the shell widget's x/y position is (0, 0) after destroying the menu bar, restore its original position.

--- a/src/xmenu.c	Tue Mar 27 09:59:39 2001 +0000
+++ b/src/xmenu.c	Tue Mar 27 14:40:34 2001 +0000
@@ -1854,6 +1854,7 @@
   set_frame_menubar (f, 1, 1);
 }
 
+
 /* Get rid of the menu bar of frame F, and free its storage.
    This is used when deleting a frame, and when turning off the menu bar.  */
 
@@ -1869,9 +1870,33 @@
   
   if (menubar_widget)
     {
+#ifdef USE_MOTIF
+      /* Removing the menu bar magically changes the shell widget's x
+	 and y position of (0, 0) which, when the menu bar is turned
+	 on again, leads to pull-down menuss appearing in strange
+	 positions near the upper-left corner of the display.  This
+	 happens only with some window managers like twm and ctwm,
+	 but not with other like Motif's mwm or kwm, because the
+	 latter generate ConfigureNotify events when the menu bar
+	 is switched off, which fixes the shell position.  */
+      Position x0, y0, x1, y1;
+#endif
+      
       BLOCK_INPUT;
+
+#ifdef USE_MOTIF
+      XtVaGetValues (f->output_data.x->widget, XtNx, &x0, XtNy, &y0, NULL);
+#endif
+      
       lw_destroy_all_widgets ((LWLIB_ID) f->output_data.x->id);
       f->output_data.x->menubar_widget = NULL;
+
+#ifdef USE_MOTIF
+      XtVaGetValues (f->output_data.x->widget, XtNx, &x1, XtNy, &y1, NULL);
+      if (x1 == 0 && y1 == 0)
+	XtVaSetValues (f->output_data.x->widget, XtNx, x0, XtNy, y0, NULL);
+#endif
+      
       UNBLOCK_INPUT;
     }
 }