On Fri, 2004-01-16 at 21:39, Adam Fedor wrote:
> > on 1/12/04 11:16 AM, NeXT at [EMAIL PROTECTED] wrote:
> > 
> > > 
> > > 1. Open GWorkspace.app and Terminal.app
> > > 2. Select GWorkspace.app for it take focus
> > > 3. Hide using menu
> > > 4. Select Terminal.app for it take focus if it's not
> > > 5. Unhide GWorkspace.app by double clicking app icon
> > > 6. The applications flickers and no menu appears and Terminal.app
> > > has focus.
> > > 
> > > I think this problem does appears with current modification in focus
> > > code.
> > >
> > 
> 
> 
> Well this patch fixes the problem for me. Perhaps you can try it:
> 
> 
> The only problem is that I have no clue why it works. At some point the
> window manager just tells the app it lost focus. I'd still like to
> figure that out.


Actually, I've settled on this patch, which makes a little more sense:


2004-02-17  Adam Fedor  <[EMAIL PROTECTED]>

        * Source/x11/XGServerWindow.m ([XGServer -orderwindow:::]): Don't
        try to raise the root window.

Index: Source/x11/XGServerWindow.m
===================================================================
RCS file: /cvsroot/gnustep/gnustep/core/back/Source/x11/XGServerWindow.m,v
retrieving revision 1.39
diff -u -r1.39 XGServerWindow.m
--- Source/x11/XGServerWindow.m 10 Jan 2004 22:41:10 -0000      1.39
+++ Source/x11/XGServerWindow.m 18 Feb 2004 04:25:43 -0000
@@ -1520,10 +1520,15 @@
        */
       if ((window->win_attrs.window_style & NSIconWindowMask) != 0)
        {
+#if 0
+         /* This doesn't appear to do anything useful, and, at least
+            with WindowMaker, can cause the app to flicker and spuriously
+            lose focus if the app icon is already visible.  */
          if (op != NSWindowOut)
            {
              XMapRaised(dpy, ROOT);
            }
+#endif
          return;
        }
       if ((window->win_attrs.window_style & NSMiniWindowMask) != 0)
_______________________________________________
Bug-gnustep mailing list
[EMAIL PROTECTED]
http://mail.gnu.org/mailman/listinfo/bug-gnustep

Reply via email to