Thanks Mattias, changes look good and work great, now merged and submitted to svn/trunk.
On 18 March 2013 08:11, Mattias Helsing <[email protected]> wrote: > Hi again > > Here is the changed source to prefer using _NET_WM_STATE_ABOVE but fall back > to XRaiseWindow should the Atoms not be supported. I'm not sure how many > might be using the raiseWindow feature but I needed it for a splash on > Ubuntu 12.04. > If you prefer I could put my change under a CMake option? > > /Mattias > > > On Mon, Mar 18, 2013 at 8:01 AM, Mattias Helsing <[email protected]> > wrote: >> >> Hi Robert! >> >> Ignore the above. My hands made a dodo and the tar is broken. Will get >> back with proper code. >> >> /Mattias >> >> >> On Sat, Jan 26, 2013 at 11:22 PM, Mattias Helsing <[email protected]> >> wrote: >>> >>> Hi Robert, >>> >>> Right you are. Modified code attached. >>> >>> And here is some background info that I found: >>> https://bugs.launchpad.net/ubuntu/+source/compiz/+bug/369954 >>> http://lists.freedesktop.org/archives/xorg/2006-January/012305.html >>> https://mail.gnome.org/archives/wm-spec-list/2003-May/msg00013.html >>> >>> /Mattias >>> >>> >>> On Fri, Jan 25, 2013 at 5:48 PM, Robert Osfield >>> <[email protected]> wrote: >>>> >>>> Hi Mattias, >>>> >>>> I've just done a quick review and don't feel comfortable with dropping >>>> the old code path complete and exclusively relying upon >>>> _NET_WM_STATE_ABOVE, what happens on older systems that don't support >>>> this? >>>> >>>> It seems more appropriate to try the Atom and if it's supported using >>>> it, otherwise fallback to the old code path. >>>> >>>> Robert. >>>> >>>> On 17 December 2012 10:28, Mattias Helsing <[email protected]> wrote: >>>> > Hi Robert, submitters >>>> > >>>> > XRaiseWindow is ignored by many modern windows managers. Instead it is >>>> > suggested that one set the _NET_WM_STATE_ABOVE hint on the window. The >>>> > submission implement just that. >>>> > >>>> > We can also put this as a hint in the GraphicsContext::Traits but I'm >>>> > not >>>> > sure if that's appropriate >>>> > >>>> > cheers >>>> > Mattias >>>> > >>>> > _______________________________________________ >>>> > osg-submissions mailing list >>>> > [email protected] >>>> > >>>> > http://lists.openscenegraph.org/listinfo.cgi/osg-submissions-openscenegraph.org >>>> > >>>> _______________________________________________ >>>> osg-submissions mailing list >>>> [email protected] >>>> >>>> http://lists.openscenegraph.org/listinfo.cgi/osg-submissions-openscenegraph.org >>> >>> >> > > > _______________________________________________ > osg-submissions mailing list > [email protected] > http://lists.openscenegraph.org/listinfo.cgi/osg-submissions-openscenegraph.org > _______________________________________________ osg-submissions mailing list [email protected] http://lists.openscenegraph.org/listinfo.cgi/osg-submissions-openscenegraph.org
