-
-
Notifications
You must be signed in to change notification settings - Fork 79.1k
em vs px media queries #22006
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
Comments
We'll be sticking with pixels given the previous discussions.
Perhaps in the future, but for now I think we should stick with what we have and get something stable in place first 😁 |
I'd go as far as suggesting that we should default to EM based media queries based on this article: https://zellwk.com/blog/media-query-units/ I'd be happy to put together a pull request with this change, or adding a variable that sets the unit for breakpoints if it was wanted. |
To add to @jasongilmour 's point right above, here is how the stock Bootstrap 4 example from the Bootstrap docs is looking like when the Font Size is set to Very Large in Chrome: That's the very first sample from here: I don't think it's right :-/ |
I looked at the bootstrap v4 source and was wondering why you are using px-based media queries, so I dug trough some issues and pull request. So I know that there has been a lot of discussion about this delicate topic (#17403, #17070), but I'd like to revisit the decision of the team.
Like some people mentioned in one the referenced issues the big benefit of em based media queries is that the layout adapts if the user has changed the base font-size, so that the content is presented in a fitting way.
I think there should be at least an variable to switch between em and px-based media queries.
The text was updated successfully, but these errors were encountered: