Re: Support for WM spec

[Owen Taylor <otaylor redhat com>]

Alexander Larsson <alla lysator liu se> writes:

> > > + * gdk_window_set_modal_hint:
> > > + * @window: A #GdkWindow
> > > + * @modal: TRUE if the window is modal, FALSE otherwise.
> > > + *
> > > + * The application can use this hint to tell the window manager
> > > + * that a certain window has modal behaviour. The window manager
> > > + * can use this information to handle modal windows in a special
> > > + * way.
> >
> > It would be nice to have a ", such as..." here to clarify the
> > behavior. but I'm not sure what that would be. I don't really
> > understand the purpose of this hint in the wmspec.
> 
> Maybe things like keeping modal windows in from of the windows they're
> modal for.

But if its transient-for, you'd think it would already being do this.
 
> > (We perhaps should wrap the ICCCM ICON_SIZES property, though,
> > if we want GtkWindow to be able to do an accurate fallback.)
> 
> Hmm? I though that the wm resized the icon window to its desired size and
> the app just rendered on expose events.

Not as far as I know. See XGetIconSizes() for more information about
ICON_SIZES.
 
> > > Index: gtk/gtkmenu.c
> > > ===================================================================
> > > RCS file: /cvs/gnome/gtk+/gtk/gtkmenu.c,v
> > > retrieving revision 1.54
> > > diff -u -p -r1.54 gtkmenu.c
> > > --- gtk/gtkmenu.c	2001/02/17 06:04:40	1.54
> > > +++ gtk/gtkmenu.c	2001/02/23 15:02:02
> > > @@ -246,6 +246,8 @@ gtk_menu_init (GtkMenu *menu)
> > >  				   NULL);
> > >    gtk_window_set_policy (GTK_WINDOW (menu->toplevel),
> > >  			 FALSE, FALSE, TRUE);
> > > +  gtk_window_set_type_hint (GTK_WINDOW (menu->toplevel),
> > > +			    GDK_WINDOW_TYPE_HINT_MENU);
> >
> > No point in this menu->toplevel is override-redirect and will
> > never be seen by the window manager.
> 
> Can you explain this a bit better? Should we use HINT_MENU on
> menu->tearoff_window then?

Window manager hints only have an effect for windows the window
manager handles.

The menu popup windows are override-redirect. Override-redirect
basically means "make this window invisible to the window manager"

Your code already seemed to be setting HINT_MENU on
menu->tearoff_window when I looked at it.

Regards,
                                        Owen