This is a very interesting article: http://blog.cloudfour.com/the-ems-have-it-proportional-media-queries-ftw/
This feels like one of those things that could become a best practice. And while I'd love it to be in the initial Zen 7.x-5.0, I fear there may be some subtle complications when browsers have different default font sizes. (The comments of the article hint at the complications.)
I think we'd need to test this pretty thoroughly before committing.
On the other hand, we're only setting the breakpoints for the default "responsive-sidebars" layout. Themers are going to need to discover the intrinsic breakpoints of their content and design, so they'll need to modify the breakpoints.
Opinions?
Comments
Comment #1
JohnAlbinA decision on this is a 7.x-5.0 blocker. Though we may decide to thoroughly test and add it to 5.1 or later.
Comment #2
LewisNymanI'm my opinion, if you don't want the calculation of breakpoints to change when the default font size if different then maybe you don't want to be using ems at all. I think it's part of the philosophy, let go of control :-)
Comment #3
MustangGB CreditAttribution: MustangGB commented@lewisnyman, except that you'd essentially be making it convenient for the themer at the inconvenience of many site viewers and still ending up with a less professional site. Can't we control the em calculations in SASS anyway?
Drop px and move to em IMO.
EDIT: How is this impacted by #1440910: Change font styling technique from "em"s to "rem"s?
Comment #4
JohnAlbinrem == em when used in media queries since they are "em"s with no parent element. Which is good because you can't use px as a fallback for media queries if you used the rem unit.
Comment #5
JohnAlbinThis is a new technique. Let's test it thoroughly and then deploy if its solid.
Removing the blocking tag.
Comment #6
caschbre CreditAttribution: caschbre commentedThis is an interesting concept and would probably require it's own mixin to accurately use and handle IE fall backs.
Comment #7
JohnAlbinThe breakpoint Sass mixin will do this automatically. Moving to 7.x-6.x
Comment #7.0
JohnAlbinUpdate issue summary
Comment #8
steinmb CreditAttribution: steinmb as a volunteer commentedComment #9
JohnAlbinWow. This totally got lost in the issue queue.
I never got a chance to play with this, so I'm just going to add a comment to _variables and people can test it out and see if it works for them. We can revisit and make it the default later.
Comment #11
JohnAlbinComment #12
JohnAlbin