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
Shows icon to disable/enable Zen Mode #12459
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.
Thank you for your pull request @ryyee99 !
Code review & Feedback
- Hide the icon in the mobile view (and possibly tablet view)
- Add a tooltip on the icon when hovered
Enter Zen mode (hide the menus)
, andExit Zen mode (show the menus)
- Show the icon in the admin screen screen as well (currently the icon is not visible in admin screen)
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.
Hi, you may want to make the styling a bit more similar to the other buttons.
Take a look at the piwikSelector
,borderedControl
and title
classes.
I'd still recommend to add a padding to the button. At the moment it's really hard to hit the button as it's only 10x16 pixel large. |
refs #9263 (comment)