Skip to content

Conversation

@badsyntax
Copy link
Contributor

This fixes source mapping when used with sass-loader.

Also, we don't want to map to the postcss generated source, we want to map to the pre-postcss source, so this is also an improvement for CSS source maps.

Inline source maps can be generated using devtool: 'inline-source-map'.

This fixes source mapping when used with sass-loader.
Also, we don't want to map to the postcss generated source, we want to map to the pre-postcss source.
ai added a commit that referenced this pull request Apr 20, 2015
@ai ai merged commit 67e68f5 into webpack:master Apr 20, 2015
@badsyntax
Copy link
Contributor Author

Thanks for the merge. This change breaks the tests though, I'm wondering if it would be better to allow configuration of map options with loader params...

@ai
Copy link
Contributor

ai commented Apr 20, 2015

@badsyntax np, I fixing test right now. Also I will move loader to PostCSS 4.1.

@badsyntax
Copy link
Contributor Author

Great, thanks a lot 👍

@ai
Copy link
Contributor

ai commented Apr 20, 2015

Released in 0.4.0 with all other PostCSS 0.4.1 features

badsyntax added a commit to badsyntax/react-seed that referenced this pull request Apr 21, 2015
@badsyntax
Copy link
Contributor Author

Thanks a lot!

@badsyntax badsyntax deleted the better-source-maps-support branch April 21, 2015 08:30
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

Successfully merging this pull request may close these issues.

2 participants