After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 330717 - STATE_ABOVE is above dock layer
STATE_ABOVE is above dock layer
Status: RESOLVED NOTABUG
Product: metacity
Classification: Other
Component: general
unspecified
Other Linux
: Normal normal
: ---
Assigned To: Metacity maintainers list
Metacity maintainers list
Depends on:
Blocks:
 
 
Reported: 2006-02-10 19:58 UTC by Rob Adams
Modified: 2006-02-11 23:58 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Clarify the code on this point (1.22 KB, patch)
2006-02-11 23:58 UTC, Elijah Newren
committed Details | Review

Description Rob Adams 2006-02-10 19:58:23 UTC
in recent metacity, marking a window "On Top" causes it to be in a layer above the dock layer.  I can't really imagine why this should be the case.
Comment 1 Elijah Newren 2006-02-10 20:22:30 UTC
Is it actually above the dock layer or at the same level?  If above, how do you duplicate?  From here, all I see is that the above windows are in the same layer -- which I also see with gnome 2.12.  Although stack.h defines them as separate levels, for some reason this appears to have been intentional according to the following ChangeLog entry:

2002-10-01  Havoc Pennington  <hp@pobox.com>

        * src/stack.c (get_standalone_layer): put ABOVE windows in same
        layer as docks.
Comment 2 Rob Adams 2006-02-10 20:34:17 UTC
actually it is the same layer; guess I just never really noticed before.  Still seems odd.
Comment 3 Elijah Newren 2006-02-10 20:46:19 UTC
Yeah, maybe Havoc could tell us the reasoning.  Havoc?
Comment 4 Havoc Pennington 2006-02-11 05:43:09 UTC
The effect of the change would be to have the focused item (dock or ABOVE window) end up on top, vs. having them in a fixed relation. (OK, stating the obvious.)

I don't remember why I did it, but if nobody has complained in 3 years and there was some reason to do it in the first place, changing it back is probably just going to cause problems... maybe there's a bug with some related discussion, you could search for bugs closed/commented around that ChangeLog entry date...
Comment 5 Elijah Newren 2006-02-11 23:58:10 UTC
Okay, I wasn't able to find a bug with relevant comments in my quick search.  However, the EWMH explicitly recommends that ABOVE and DOCK should be in the same layer, which may have been the reason for the change.

I do think it'd be worthwhile to clean up the code to make this more clear.  I'll attach a patch in a minute that I'll commit which does so.
Comment 6 Elijah Newren 2006-02-11 23:58:38 UTC
Created attachment 59160 [details] [review]
Clarify the code on this point