Remove the need to add autoprefix on development - Fix #85

moxie-lean / ng-web-app

Front-end component of a web app based on the modular LEAN concept.

http://getmoxied.net/lean

MIT License

6 stars 0 forks source link

Closed mitogh closed 8 years ago

mitogh commented 8 years ago

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

Feature

More information: #85

More information: #85

Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)

adamf321 commented 8 years ago

A few considerations we should address before merging this:

We should support both FF and Chrome as this tool is open source
I think some CSS attributes require prefixes in Chrome too, often new ones before they are fully implemented in the standards.
Could we lose time debugging issues which only happen in staging but not local? Like we see now with missing $inject dependencies during minification.

mitogh commented 8 years ago

Ok.

Both are considered.
Prefixes are used for older versions where the rules were not on the standard at that point, and we shouldn't use a property or a CSS rule that only works on one browser since it's not going to work on browsers that are older and we support.
Only if you are using old versions of browsers (talking in years)
- Plus the autoprefixer is present during the build and not on the development phase.