sindresorhus / gulp-autoprefixer

Prefix CSS
MIT License
694 stars 50 forks source link

-ms prefix not added to transform #14

Closed renestalder closed 10 years ago

renestalder commented 10 years ago

As in my test, using autoprefixer with the following settings, it seems not to generate the transform attributes with the -ms prefix.

autoprefixer('last 2 version', 'safari 5', 'ie 9', 'opera 12.1', 'ios 6', 'android 4')

Raw code:

.tile__front {
  transform: rotateY(180deg);
}

What autoprefixer outputs

.tile__front {
  -webkit-transform: rotateY(180deg);
  transform: rotateY(180deg);
}

According to caniuse, the -ms prefix is needed for IE9.

renestalder commented 10 years ago

Update core first before test if it's actually a core problem which might be fixed.

battaglr commented 10 years ago

Any bug with autoprefixer itself should be reported there. Thanks!

jacksaunders commented 8 years ago

I am getting the exact same issue described above, i have tested on https://autoprefixer.github.io/? and it is working correctly on there, however the generated CSS from gulp-autoprefixer is missing the -ms tag.

AP settings

browsers: [
   'last 2 versions',
   'ie 8',
   'ie 9', 
   'android 2.3',
   'android 4',
   'opera 12'
  ]

Sass code

&.-has-sub-list.-is-closed .filter__arrow {
    transform: rotate(-90deg);
    transition: transform 0.2s linear;
  }

Generated CSS

.filter__single .filter__list li.-has-sub-list.-is-closed .filter__arrow {
  -webkit-transform: rotate(-90deg);
  transform: rotate(-90deg);
  -webkit-transition: -webkit-transform .2s linear;
  transition: transform .2s linear
}
zapatoche commented 8 years ago

Hi there,

I have the same issue.

battaglr commented 8 years ago

@zapatoche, are you using the same config in both cases?

zapatoche commented 8 years ago

Yop, I even simplified it to just 'ie >= 9' to make sure there wasn't any interference.

battaglr commented 8 years ago

@zapatoche, have you checked that the versions of autoprefixer are the same —both in this plugin as in the CLI?

zapatoche commented 8 years ago

Amazeballs!!! the latest version wasn't installed despite running npm update, i'm now on 3.1.1 and it works like a charm. Thanks