jnsh / arc-theme

A flat theme with transparent elements (actively maintained fork)
GNU General Public License v3.0
900 stars 77 forks source link

1px vertical shift between desktop view and activities overview #124

Closed xoan closed 3 years ago

xoan commented 3 years ago

Details

Some elements in top bar (status icons, weather in clock, ...) have a 1px vertical position variation between desktop view and activities overview.

Captura del escritorio de 14-04-21 09:49:52

jnsh commented 3 years ago

Sorry but I can't reproduce this.

Can you try whether this still occurs if you disable all your extensions? If that helped, re-activate the extensions one-by-one, to check which one results in this issue.

xoan commented 3 years ago

I have the same behavior with all the extensions disabled (unless User Themes) and even by hidding battery percentage.

Captura del escritorio de 14-04-21 10:28:41

The clock button is not affected, but the activities button (three dots in this shell theme) has the same bug as status icons.

I'm using the -solid variant [1] in Arch Linux repositories with Papirus icons, but also tested with Adwaita.

[1] https://archlinux.org/packages/community/any/arc-solid-gtk-theme/

jnsh commented 3 years ago

Thank you for testing.

After some more digging, I figured out this depends on the font scaling factor: With 0.9 (which is what I've been using) there are no problems, but e.g. 0.8 or 1.0 the panel elements get jumpy..

Thank you for reporting this. I'll try to figure out a solution soon.

liolok commented 3 years ago

It seems like a shift at first, but I noticed it's actually a vertical shrink. Workspace indicator would be clearer to see the change.

Also tested on arc-gtk-theme and it behaves the same.

jnsh commented 3 years ago

I'm still trying to figure out the best solution to this, but reverting https://github.com/jnsh/arc-theme/commit/775abec762fdf89b236a1ee3168459d5af0b3932 should fix this if anyone wants to work around the issue locally.

jnsh commented 3 years ago

This should now be fixed after reverting the commit mentioned above.

Closing.