-
-
Notifications
You must be signed in to change notification settings - Fork 755
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
change opacity of sidebar toggle button #4132
Conversation
✅ Deploy Preview for volto ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
Test summaryRun details
View run in Cypress Dashboard ➡️ This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard |
Some time ago on a client project we added an animation which flashed the sidebar toggle on entering the page, to draw attention to it. @sneridagh would that be a good idea here? |
@tiberiuichim, I think that part is already implemented like when you refresh the page, sidebar handle will blink for 3 times. |
Right, I forgot that we merged that one :) |
To draw more attention to the sidebar toggle we can add tilting and poping up(little bit) animation to it everytime page loads. |
…ty' into sidebarHandlerOpacity
Blinking does work , but changing its color seems better idea -> like the default grey or blue |
Hi @aadarsh-nagrath, this PR is closed but you can check issue #3910 related to this and the already existing WIP PR #3912 . |
fixes #4131
In "theme/themes/pastanaga/globals/site.variables" changed variable
sidebarToggleButtonNormalOpacity:
from 0.05 to 0.3 and when on hover state changed opacity from 0.3 to 0.5 which makes it more noticeable and improves UX.Demo
Untitled.video.-.Made.with.Clipchamp.5.mp4