Skip to content
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

Drop -webkit-box-sizing #12640

Closed
wants to merge 1 commit into from
Closed

Drop -webkit-box-sizing #12640

wants to merge 1 commit into from

Conversation

BBosman
Copy link
Contributor

@BBosman BBosman commented Feb 7, 2014

All supported browsers support it without the -webkit prefix, so we should drop it.

http://caniuse.com/css3-boxsizing

@cvrebert
Copy link
Collaborator

cvrebert commented Feb 7, 2014

20% of Android users are still on v2.3 "Gingerbread", whose non-Chrome stock browser requires the prefix. The Android stock browser is indeed not officially supported, but I still fear the number of issues that people might file as a result of this change.

@cvrebert cvrebert added the css label Feb 7, 2014
@zlatanvasovic
Copy link
Contributor

@cvrebert 20% of Android users are still on Gingerbread, but 19.99% of those users (including me, but I have also newer phone) migrated to Firefox, Opera or something else. 😉

Bigger problem is Safari 5, on older Macs.

@BBosman
Copy link
Contributor Author

BBosman commented Feb 7, 2014

Normalize.css doesn't use any of the prefixes that 2.3 requires and there are places in the bootstrap code itself where we're not using them (box-shadow, background-clip and background-size need to be prefixes for example).

So support for 2.3 is already flaky at best.

@mdo
Copy link
Member

mdo commented Feb 7, 2014

Indeed we don't have -webkit prefixes for many of our visual CSS properties like box-shadow. However, the box model is a much more important thing to keep intact for the maximum number of users, no matter what. We don't officially support Android 2.x, but we do everything we can to help.

For now, we'll leave it.

@mdo mdo closed this Feb 7, 2014
@BBosman BBosman deleted the box-sizing branch February 7, 2014 23:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants