-
Notifications
You must be signed in to change notification settings - Fork 292
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
runfix: adress responsivness of the preferences sections (WPB-7214) #17409
Conversation
Quality Gate passedIssues Measures |
input { | ||
color: var(--background); | ||
} |
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.
This appears to be a years old leftover that we can remove
overflowX: 'hidden', | ||
|
||
// Rely on viewport media queries if container queries are not supported by the browser | ||
'@media (max-width: 768px)': { |
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.
css variables are not supported in media queries and less variables don't work outside of less files unfortunately
Description
We can use container queries to have the content of the preference pages be responsive with its container instead of the viewport.
Standard media queries are used as a fallback.
This allows the preferences page to be responvive regardless of the state of the new sidebar
Screenshots/Screencast (for UI changes)
Before: (layout breaks just before going into responsive design with the sidebar opened)
After: