Problem w change in media query processing about react-styled-flexboxgrid HOT 8 CLOSED

Hi
First, thanks for your feedback.
The motivation behind #13 has not been explain and it's a shame ;) I didn't handle it directly so I can't explain clearly why we did this. I wasn't confortable with the fact to go to pixels values too.

I will come back here next week with the explanation and we will discuss about it.

from react-styled-flexboxgrid.

Perhaps you could add the breakpoint unit type to the theme definition so that people can customize it, if they want that, but otherwise functionality is preserved

from react-styled-flexboxgrid.

As illustrated in the following article, em should be used in breakpoints to work properly cross-browser and support users changing their browsers font-size

https://zellwk.com/blog/media-query-units/

from react-styled-flexboxgrid.

If you read the comments in that article, you'll see evidence from folks that the author is incorrect in his conclusions - specifically the Safari not reporting px the same as the other browsers when zooming. If you look at the animated gif in his article, you'll see that the px test matches the other browsers' tests; its the em/rem tests that differ on Safari. Px based queries get consistent results across browsers when zooming.

I just tested this on Chrome and Safari and both Chrome and Safari's media query with px behave the same. You can try with http://codepen.io/blyne/pen/mPwmvX (I changed from the 400px/25em/25rem to 640px/40em/40rem)

from react-styled-flexboxgrid.

Good point @zentuit , i have overlooked the comments.

A media query using em or rem (e.g. max-width) is based on the size of the viewport.
The viewport isn't affected by the elements inside of it even if the elements are larger than the viewport.
CSS gives us control over the font sizes of the elements, but not the viewport.

I guess that would explain the switch from em to px ?

from react-styled-flexboxgrid.

Let's also not forget that this library is supposed to be a react implementation of flexboxgrid, which clearly goes the "em" way

from react-styled-flexboxgrid.

Thanks for creating this lib, I think it's really easy to use. Is there any plan to deal with this issue?

from react-styled-flexboxgrid.

Sorry to being back late.
@mschipperheyn @splars I agree with you, this library is supposed to be kind of port of flexboxgrid and I want to follow their implementation.

I reverted the commit in #35 and it is published v1.1.0

Thanks

from react-styled-flexboxgrid.