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 738800 - Menu border missing in Adwaita when no compositor is running
Menu border missing in Adwaita when no compositor is running
Status: RESOLVED FIXED
Product: gtk+
Classification: Platform
Component: Themes
3.14.x
Other Linux
: Normal normal
: ---
Assigned To: gtk-bugs
gtk-bugs
Depends on:
Blocks:
 
 
Reported: 2014-10-19 11:22 UTC by Tobias Getzner
Modified: 2014-11-03 03:09 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Screenshot (21.53 KB, image/png)
2014-10-19 11:29 UTC, Tobias Getzner
  Details
adds border in a non composited env (3.07 KB, patch)
2014-10-22 18:51 UTC, Lapo Calamandrei
none Details | Review
screenshot with metacity (39.16 KB, image/png)
2014-10-23 03:22 UTC, Matthias Clasen
  Details
Second attempt (3.72 KB, patch)
2014-10-23 11:46 UTC, Lapo Calamandrei
committed Details | Review
screenshot with openbox (26.59 KB, image/png)
2014-10-23 11:51 UTC, Matthias Clasen
  Details

Description Tobias Getzner 2014-10-19 11:22:10 UTC
As of GTK/Adwaita 3.14, all menu panes (context menu, pull-down menu, etc.) are missing a border around the pane, so that the menu blends directly into white backgrounds below.

I have noticed that this issue disappears when a compositor is being run.
Comment 1 Tobias Getzner 2014-10-19 11:29:45 UTC
Created attachment 288841 [details]
Screenshot
Comment 2 Lapo Calamandrei 2014-10-20 09:24:56 UTC
True, we want to use a shadow istead of border in a composited env since it gives better results, we should treat the other case with a border, not sure I have a way to do that by the theme btw
Comment 3 Matthias Clasen 2014-10-22 10:45:54 UTC
I believe when I added the csd support in menus, the differentiation between

.window-frame.csd.popup

and

.window-frame.popup

worked for this.
Comment 4 Lapo Calamandrei 2014-10-22 14:19:21 UTC
Border doesn't seem to work there tho :-/

try .window-frame.popup { border: 1px solid red; } within the inspector, even combining it with .menu doesn't seem to work
Comment 5 Lapo Calamandrei 2014-10-22 18:51:11 UTC
Created attachment 289166 [details] [review]
adds border in a non composited env

This should fix that, I have no ways to try it atm, can you guys confirm it works right? It's against master, but I it should apply to 3.14 as well
Comment 6 Matthias Clasen 2014-10-23 03:22:28 UTC
Created attachment 289185 [details]
screenshot with metacity
Comment 7 Lapo Calamandrei 2014-10-23 11:46:35 UTC
Created attachment 289198 [details] [review]
Second attempt

Let's try resetting the shadow
Comment 8 Matthias Clasen 2014-10-23 11:51:01 UTC
Created attachment 289199 [details]
screenshot with openbox
Comment 9 Matthias Clasen 2014-10-23 11:54:09 UTC
Tobias, what window manager is your screenshot taken with ?
Comment 10 Lapo Calamandrei 2014-10-25 12:38:08 UTC
I guess this is a general problem, since we're using shadows as borders in an enviroment where we can control them (Gnome), anywhere else we don't know, thinking about other platforms too, hence I guess we need a proper solution.
Any idea Benjamin?
Comment 11 Benjamin Otte (Company) 2014-10-25 21:02:59 UTC
Wasn't the idea that we draw shadows ourselves on menus just like we do on regular windows?
Comment 12 Matthias Clasen 2014-10-27 12:57:54 UTC
(In reply to comment #11)
> Wasn't the idea that we draw shadows ourselves on menus just like we do on
> regular windows?

the problem is one of control and communication - some compositors decide to draw shadows on menus themselves (metacity is an example), and some even expose this in their user preferences (kwin, I believe). Hard to make anything that works everywhere, with this much variation.
Comment 13 Tobias Getzner 2014-10-28 10:46:13 UTC
(In reply to comment #9)
> Tobias, what window manager is your screenshot taken with ?

This was on Qtile. I’m not seeing the borders in any WM, though, unless
an additional compositor is running. I just tried it with dwm and twm to be
certain.