Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Smooth switch between dark and light styles #354

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

teamcons
Copy link

Just a suggestion :)
The dock adapts in a smooth manner when changing style in system settings: slowly gets darker if switch to dark, slowly gets lighter if switch to light the animation follows the same as wingpanel becoming opaque - and as in several places in the stylesheet

the animation is slower than wingpanel - as that one is too short and like current behaviour: immediate blink. I would go with slower here because the user is being eased in to the new colour - It is not the same context and intent as the interface adapting to a change

Just a suggestion :)
The dock adapts in a smooth manner when changing style in system settings: slowly gets darker if switch to dark, slowly gets lighter if switch to light
the animation follows the same as wingpanel becoming opaque - and as in several places in the stylesheet

the animation is slower than wingpanel - as that one is too short and like current behaviour: immediate blink.
I would go with slower here because the user is being eased in to the new colour - It is not the same context and intent as the interface adapting to a change
@teamcons
Copy link
Author

teamcons commented Jan 31, 2025

i summon @elementary/desktop-developers !
(i didnt know i had to do it)

@stsdc stsdc requested a review from a team January 31, 2025 23:00
@@ -5,6 +5,7 @@

dock {
background: @bg_color;
transition: background 800ms cubic-bezier(0.4, 0, 0.2, 1);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Probably should also transition box-shadow

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same effect ?
how is the duration for you ?

i would like to suggest the same transition for the rest of eOS stylesheet, but i yet am not sure yet how to target the components efficiently so as to avoid having the transitions at any other time than when switching style

in a later PR probably.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants