Skip to content
This repository has been archived by the owner on Aug 30, 2018. It is now read-only.

Update prefixing mixins #407

Closed
benknight opened this issue May 8, 2015 · 4 comments
Closed

Update prefixing mixins #407

benknight opened this issue May 8, 2015 · 4 comments

Comments

@benknight
Copy link
Contributor

The prefix mixins are doing a bit of unnecessary work:

  1. transition no longer needs prefixing at all. http://caniuse.com/#search=transition
  2. Transforms don't need -o- or -moz- prefixing: http://caniuse.com/#search=transform
  3. Linear gradients no longer need prefixing really, except maybe -webkit- for super deep mobile browser support? http://caniuse.com/#search=transform
@benknight
Copy link
Contributor Author

Happy to submit a pull request if authors agree. :)

@cshold
Copy link
Contributor

cshold commented May 11, 2015

Agreed, I'll 👍 a PR. I was actually thinking of removing the gradient mixin. It's not used anywhere, and I've very rarely been wanting one while developing.

@graygilmore
Copy link
Contributor

Looks like this has been taken care of? 3ab33cb

@cshold
Copy link
Contributor

cshold commented Jun 9, 2015

Fixed via #410

@cshold cshold closed this as completed Jun 9, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants