-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Define Pure's browser support #36
Comments
@ericf I mentioned the browser support in #26. I agree that we should outline this on the home page of the site. This should be an issue on https://github.com/yui/pure-site/ rather than here. |
I think it should be in the README.md too. |
Agreed this should not only be on the home page of the site, but also in this repo's README.md or possibly in a CONTRIBUTORS.md. The reason to add the info here is more around setting expectations for what is acceptable in a PR. |
Good call @evangoer |
It would love to see a PureCSS alternative version supporting only IE8+(as normalize does). I think that most of websites being built nowdays are targeted for IE8+. Besides, it would make PureCSS even smaller size :). |
@CharliePops The code needed to support IE7 is pretty minimal. I doubt it would make much of a difference once gzipped. Pure uses Normalize 1.x, which supports legacy browsers such as IE6 and 7. Since it's just CSS, I think it makes sense to support older browsers. That's where most of this layout help is needed anyway. Modern browsers are pretty good when it comes to CSS. Also, just because we support legacy browsers doesn't mean we can't use the sexy new CSS3 properties. They all gracefully degrade on older browsers. |
Added this info to the README.md. Closing this out. |
Pure should have explicitly defined browser support so people know what to expect it will work on when using it.
Also, I think it things like when vendor prefixes should be used need to be defined. e.g. Do we need to use vendor prefixes on rules which modern browsers (that auto update) now support without prefixes, because people may be using older versions that need the prefix?
The text was updated successfully, but these errors were encountered: