ritwickdey / vscode-live-sass-compiler

Compile Sass or Scss file to CSS at realtime with live browser reload feature.
https://ritwickdey.github.io/vscode-live-sass-compiler/
MIT License
657 stars 169 forks source link

Incorrect generation of map files #70

Open sakralbar opened 6 years ago

sakralbar commented 6 years ago

Mismatched rows with classes in css 1529670536122 1529670563967

ritwickdey commented 6 years ago

That is the issue? .navBar is showing correctly. & what is the name of the source file?

ritwickdey commented 6 years ago

Ohh! Sorry! I was confused!!

I'm not sure why! If you turn off auto prefix, this is working?

sakralbar commented 6 years ago

You're right. If you disable the auto prefix, the line numbers of css-classes are displayed correctly. If the auto prefix is enabled, then a shift of the rows to some random number is observed. Sorry if this is not an issue.

Steve-VE commented 6 years ago

Hello, I have the same problem since the last update (before this update, I didn't use the auto prefix, but it's was activated by default after the update). When I disabled the auto prefix, it's working correctly.

ritwickdey commented 6 years ago

Yep! I think it's a bug. I'll check it

cjarit commented 6 years ago

Same problem here. For a while though.

imike57 commented 6 years ago

Same for me... Still no solution ?

boyum commented 6 years ago

I've tried working on this, but haven't found a solution yet. Reading from the postcss source maps guides, it seems apparent that we'll need to build the files first (non-autoprefixed), before building new source maps after autoprefixing.

Any thoughts, @ritwickdey?

ico3iain commented 5 years ago

@ritwickdey - is there any development on a fix for this?

Thanks

TakahashiEi commented 5 years ago

Same for me.

i-am-malaquias commented 5 years ago

Having the same issue but turning off auto-prefix isn't an option for me.

nepomuk88 commented 5 years ago

Same Issue, i'd be pleased for a fix

vortexawy commented 5 years ago

I think we should disable auto prefix option and use another plugin for prefixing like postcss autoprefixer to apply to the main CSS file at the end of developing

Sam-Smyth commented 4 years ago

Has there been any resolution to this? Having to choose between the auto prefix option and having source maps work properly is a pain. @ritwickdey

Christopher-Hayes commented 3 years ago

@glenn2223 is continuing work on this extension in a new repo at: https://github.com/glenn2223/vscode-live-sass-compiler That code includes a fix for this bug. If you search for "Live SASS Compiler - Glenn Marks", you can test out the preview extension. Note that this extension uses Dart SASS which will have the latest SASS specs, but may run slightly different than from deprecated Node SASS.

TonyTheFerg commented 3 years ago

+1 having the same issue.

glenn2223 commented 3 years ago

@TonyTheFerg, as per this comment (above), have you tried my extension?

This one here is no longer being maintained, whereas mine is (by me 😁). You can find a guide to migrating - the last FAQ - here (just change a couple of settings, if you use them)

TonyTheFerg commented 3 years ago

@glenn2223 Thanks! That worked. How do I turn on showOutputWindowOn?

Got it with this: "liveSassCompile.settings.showOutputWindowOn": "Debug",

glenn2223 commented 3 years ago

@glenn2223 Thanks! That worked.

Good, I'm glad 👍😁

How do I turn on showOutputWindowOn?

Got it with this: "liveSassCompile.settings.showOutputWindowOn": "Debug",

Glad you got it working. I was just about to send you this link, which has details on what each setting will output