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

Remove unneeded prefixes from some mixins #8380

Merged
merged 1 commit into from
Jul 18, 2013
Merged

Conversation

joeybaker
Copy link
Contributor

Per caniuse, prefixes for transitions and transforms can be reduced to just -webkit and -ms. http://caniuse.com/#search=transform
http://caniuse.com/#search=transition

Per caniuse, prefixes for transitions and transforms can be reduced to just `-webkit` and `-ms`. http://caniuse.com/#search=transform
http://caniuse.com/#search=transition
@mdo
Copy link
Member

mdo commented Jul 5, 2013

How far back does -moz go with Firefox? And are the -o ones gone since Opera supports all the -webkit versions? We'll want to clarify our browser support with v3, so let's be super specific here.

@cvrebert
Copy link
Collaborator

cvrebert commented Jul 6, 2013

Between MDN's compatibility tables & some googling:

  • transition-* & transform were made unprefixed in Firefox 16.0 and Opera 12.50
  • background-size (w/o SVG support) & background-clip were unprefixed in Firefox 4.0
  • background-size was unprefixed in Opera 10.0
  • FWICT, Blink-based Opera (mobile v14+, desktop v15.0+), like Blink generally, will support -webkit in a legacy fashion and will not support -o

@joeybaker
Copy link
Contributor Author

@cvrebert has it pretty well covered. Basically, this represents every current version of browsers, +2 versions back, and every browser with a usage base greater than 1%.

mdo added a commit that referenced this pull request Jul 18, 2013
Remove unneeded prefixes from some mixins
@mdo mdo merged commit 02bddac into twbs:3.0.0-wip Jul 18, 2013
@mdo
Copy link
Member

mdo commented Jul 18, 2013

Thanks guys! <3

mdo added a commit that referenced this pull request Jul 18, 2013
@mdo mdo mentioned this pull request Jul 18, 2013
@joeybaker joeybaker deleted the patch-1 branch July 18, 2013 15:33
@awebdeveloper
Copy link
Contributor

samething can be done for gradient http://caniuse.com/#search=grad

Except for webkit we can remove all other prefix

@cvrebert
Copy link
Collaborator

@awebdeveloper Open a new issue or pull request.

stempler pushed a commit to stempler/bootstrap that referenced this pull request Apr 11, 2014
Remove unneeded prefixes from some mixins
stempler pushed a commit to stempler/bootstrap that referenced this pull request Nov 4, 2014
Remove unneeded prefixes from some mixins
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