-
-
Notifications
You must be signed in to change notification settings - Fork 188
Add rem fallback #31
Comments
Ref #32 |
why shouldn't it be the default behaviour? |
This is adding a lot of new rules. And it's only required for IE 8-. |
it's not that much rules, is it? btw IE8 is actually the lowest browser many people still have to care about (i.e. http://www.quirksmode.org/blog/archives/2014/09/ie_survey_resul.html) |
It can be a lot of if you use rem a lot. For my concern it's the unit I use the most now (maybe it's not a good thing). So it can easily create between x1.5 to x2 rules. |
yup |
Probably not worth the time |
I've take a quick look to add this feature (disabled by default) using node-pixrem (based on postcss), but the plugin is not exported & use a global option (ouch). This will need a PR to make things right. |
It's only IE8 and Opera Mini without |
I know, but I still have projects for client like Airbus that need IE8 support (mainly because they do not update their IT assets). |
Sure. But it can just be a postcss plugin right? Does it have to be included in this project? |
I think according to the way I load plugins (by using require() only when needed), it won't create any perf issue if it's defined as dep, but not required all the time. |
Now that #32 has landed in master, I will add rem and it might be disabled by default according to browserslist default. |
Not sure if this should be the default behavior.
We need make something like autoprefixer to select browser support.
The text was updated successfully, but these errors were encountered: