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

Enable CSS sourcemaps in development #590

Closed
JaKXz opened this issue Sep 5, 2016 · 8 comments
Closed

Enable CSS sourcemaps in development #590

JaKXz opened this issue Sep 5, 2016 · 8 comments

Comments

@JaKXz
Copy link

JaKXz commented Sep 5, 2016

Hi! Firstly, of course, thank you for this amazing project! 👏

I've seen the comments on sourcemaps: #109, #139, #494, #343, etc. but didn't see one about enabling CSS sourcemaps. I thought I'd make a PR first but I wanted to discuss first and get to grips with this repo structure.

Since there aren't breakpoints to worry about, this could be a very simple enhancement, e.g:

loader: 'style-loader!css-loader?'.concat(isProduction ? 'minimize' : 'sourceMap')

where const isProduction = process.env.NODE_ENV === 'production'. Apologies if I am duplicating / creating noise, I thought I'd searched thoroughly.

@gaearon
Copy link
Contributor

gaearon commented Sep 5, 2016

Sounds perfectly reasonable. I would expect Webpack to already do it in DEV mode, does it not? Do you know why?

@JaKXz
Copy link
Author

JaKXz commented Sep 5, 2016

This is what I'm seeing ATM but I couldn't tell you why it's happening other than the config not being in place:

screenshot 2016-09-05 16 04 23

@mareksuscak
Copy link
Contributor

mareksuscak commented Sep 11, 2016

@JaKXz FYI: Relative paths in source maps are buggy and you need to use an absolute public path which includes the server URL. Read more here.

Also, shall we use eval-source-map or cheap-eval-source-map devtool in development? Makes debugging a bit easier with the original source code.

EDIT: Let me know if you need help.

@gaearon
Copy link
Contributor

gaearon commented Sep 30, 2016

Going to close since #591 didn’t work out, and I don’t know how else to implement this.

@gaearon gaearon closed this as completed Sep 30, 2016
@JaKXz
Copy link
Author

JaKXz commented Sep 30, 2016

@gaearon it should probably be left open in that case... right?

@gaearon
Copy link
Contributor

gaearon commented Sep 30, 2016

If you know of any possible alternative solution, sure. I’m just not aware of any.

@skaltenegger
Copy link

skaltenegger commented Apr 20, 2017

I got CSS sourcmaps and hot reloading working by following @mareksuscak's advice of changing my css loader in the webpack config to the following:

{ test: /.css$/, loader: 'style!css?sourceMap!postcss' },

And I can't complain about FOUC (flash of unstyled content) - so all good for development!

Hope that may help someone here! 😎

@pastinepolenta
Copy link

as @skaltenegger wrote the config for webpack 2 is

{
    test: /\.css$/,
    use: [
         require.resolve('style-loader'),
         {
             loader: require.resolve('css-loader'),
             options: {
                 importLoaders: 1,
                 sourceMap: 1
             },
          },
          ....

@lock lock bot locked and limited conversation to collaborators Jan 21, 2019
This issue was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants