Skip to content
This repository has been archived by the owner on Jul 8, 2020. It is now read-only.

autoprefixer seems to be broken in 1.3.2 #251

Closed
MathiasTim opened this issue Sep 16, 2015 · 1 comment
Closed

autoprefixer seems to be broken in 1.3.2 #251

MathiasTim opened this issue Sep 16, 2015 · 1 comment
Labels
Milestone

Comments

@MathiasTim
Copy link
Contributor

I just found out, that the autoprefixer is broken with the current configuration.

I managed to change a few lines so it works now, but I'm not sure about it.
I commented the Filter out and moved the autoprefixer before the sourcemaps.write

Here is my code:

// build styles to tmp
gulp.task('styles', ['clean'], function () {
  // var cssFilter = $.filter(['*.css', '!*.map'], {restore: true});

  // compile css starting from each module's scss
  return gulp.src('app/*/styles/!(_)*.scss')
    .pipe($.plumber())
    .pipe($.sourcemaps.init())
    .pipe($.sass.sync().on('error', $.sass.logError))
    .pipe($.autoprefixer({ browsers: ['last 2 versions'], remove: false}))
    .pipe($.sourcemaps.write('.'))
    // .pipe(cssFilter)
    // .pipe(cssFilter.restore)
    .pipe(gulp.dest('.tmp/'));
});
@MathiasTim MathiasTim added the bug label Sep 16, 2015
@gruppjo
Copy link
Contributor

gruppjo commented Sep 16, 2015

fix according to above fix and this: https://www.npmjs.com/package/gulp-autoprefixer#source-maps

gruppjo added a commit that referenced this issue Sep 16, 2015
1.3.3

* tag '1.3.3':
  1.3.3
  fixes #251: autoprefixer seems broken
  fixes #254: eslint adaptions
  fixes #253: remove eslint errors in injecting.js
  fixes #252: linting-throw not working as expected
gruppjo added a commit to mwaylabs/generator-m-ionic-demo that referenced this issue Sep 16, 2015
@gruppjo gruppjo modified the milestone: 1.4.0 Sep 17, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

2 participants