Gala: SSD windows have very strong shadows

Created on 26 Oct 2018  路  3Comments  路  Source: elementary/gala

A minor issue, but it's not really pleasant on the eyes. This is the drop shadow of System Settings, which is fine:

eos-system-settings

But with Slack:

eos-snap-slack

Or with Gravit Designer:

eos-snap-gravit

Shadows are really intense, too intense for my taste.


Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

Most helpful comment

The stylesheet fix has landed, now it needs to be fixed in Gala.

While shadow gsettings keys indeed need to be removed, they are handy for playing with the shadows right now.

Screenshot from 2019-11-04 16-44-47

Screenshot from 2019-11-04 16-43-21

What about these vaues? It can't look 100% same because, well, the shadow is calculated differently, but it can be adjusted to look closer. The left window shows the value + the reference CSD shadow, the right one demoes the SSD shadow.

CC @danrabbit

All 3 comments

Screenshot from 2019-10-23 08-52-39

It's not because of Snap, but because those apps are using legacy decorations.

Here gala (well, mutter) assumes that GTK stylesheet only specifies a border in .ssd decoration style. See this line in Adwaita: https://gitlab.gnome.org/GNOME/gtk/blob/gtk-3-24/gtk/theme/Adwaita/_common.scss#L4497

elementary stylesheet doesn't do this, so mutter draws the default shadow + full box-shadow from stylesheet, so there are 2 shadows on top of each other.

The stylesheet fix has landed, now it needs to be fixed in Gala.

While shadow gsettings keys indeed need to be removed, they are handy for playing with the shadows right now.

Screenshot from 2019-11-04 16-44-47

Screenshot from 2019-11-04 16-43-21

What about these vaues? It can't look 100% same because, well, the shadow is calculated differently, but it can be adjusted to look closer. The left window shows the value + the reference CSD shadow, the right one demoes the SSD shadow.

CC @danrabbit

@Exalm that's a massive improvement. Nice work! Happy to merge that

Was this page helpful?
0 / 5 - 0 ratings