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

Sass sourcemaps #64

Open
djmtype opened this issue Sep 23, 2014 · 3 comments
Open

Sass sourcemaps #64

djmtype opened this issue Sep 23, 2014 · 3 comments

Comments

@djmtype
Copy link

djmtype commented Sep 23, 2014

I installed Fireshell, but also updated the node modules. With that said, Sass along side grunt-contrib-sass now support sourcemaps. I have used it with Grunt successfully in other projects. However, I noticed Fireshell's setup does not coordinate properly. For example, if I declare a style in _app.scss, Web Inspector is telling me the style is found in _print.scss when it's clearly not.

Any idea what may be the cause?

@jonlow
Copy link

jonlow commented Sep 24, 2014

What does your grunt file look like?

@djmtype
Copy link
Author

djmtype commented Sep 24, 2014

I have not touched the gruntfile that comes with Fireshell, so my gruntfile would be the default. Also, I tried without updating the modules and get the same results. If you run Fireshell out of the box are you not experiencing the same thing? Try writing to one of your SCSS partials, inspect it and see if it matches Web Inspector.

I don't think there is an issue with grunt-contrib-sass because I'm currently using it with a Roots-Sass project and my SCSS sourcemaps coordinate just fine, however, Fireshell and Roots-Sass config is not the same, I realize.

@djmtype
Copy link
Author

djmtype commented Sep 25, 2014

Based on my recent discovery, it should include the map option in the autoprefixer task.

// Example from another gruntfile that outputs sass sourcemaps to Web Inspector

autoprefixer: {
  multiple_files: {
    expand: true,
    flatten: true,
    src: 'sass/*.scss, sass/**/*.scss',
    dest: 'css/'
  },
  sourcemap: {
    options: {
      map: true
    },
    src: 'css/styles.css',
    dest: 'css/styles.css'
  }
}

Next step—compare the above syntax with Fireshell's implementation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants