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
Update dark mode toggle switch #2588
Update dark mode toggle switch #2588
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm seeing a slight delay from when I click the button to when the animation starts. Is this delay intentional?
tmp.mp4
@lukebp the current delay time for animation effect is 200 mili second. While the change of the variable is immediately. May be I should change the animation effect to 100, so the time seems to fit better. Or we can update the variable after the animation was done. So we will see the change of background after the button animation process |
What about taking out the delay entirely? It doesn't look like there is one on test-proposals. tmp-2021-09-17_07.24.42.mp4 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tACK on firefox
Needs code approval from @tiagoalvesdulce.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
height: 100%; | ||
display: flex; | ||
align-items: center; | ||
} | ||
|
||
.navContainer { | ||
composes: sharedNavAtribute; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
good job using composes!
Depended on decred/pi-ui#370. |
close #2497
This pull request update the dark mode toggle to be a new designed
toggle. The preview of the toggle here:
It also does some adjustments with nav header style follow the comment
The preview behavior can be seen in the below video:
vokoscreen-2021-09-16_23-24-38.mp4