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
LewisNyman CreditAttribution: LewisNyman commentedI'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