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

Source maps doesn't work correctly (node-sass from CLI) #834

Closed
cachaito opened this issue Apr 4, 2015 · 3 comments
Closed

Source maps doesn't work correctly (node-sass from CLI) #834

cachaito opened this issue Apr 4, 2015 · 3 comments

Comments

@cachaito
Copy link

cachaito commented Apr 4, 2015

Hi!
Tested environment: Win7 64bit, Node 0.12.2, node-sass 2.1.1

Code from Npm scripts CLI:

 "scripts": {
    "pretask-libsass": "node-sass --source-map true --output-style expanded scss/custom.scss .tmp/css/custom.css",
    "task-libsass": "node-sass --watch sass -o .tmp/css --source-map .tmp/css/custom.css.map --output-style expanded scss .tmp/css/custom.css"
    "start": "npm run task-libsass"
  }

I've noticed that when I DevTools inspect styles from imported *.scss files, it always pointing to the main custom.scss instead of _headers.scss or _typography.scss

@cachaito cachaito changed the title Souce maps doesn't work correctly (node-sass from CLI) Source maps doesn't work correctly (node-sass from CLI) Apr 4, 2015
@am11
Copy link
Contributor

am11 commented Apr 6, 2015

This issue belongs to LibSass. node-sass does not alter source-maps generate by LibSass.
LibSass recently fixed some source-map related issues, you can retest your usecase using v3 beta.

@cachaito
Copy link
Author

cachaito commented Apr 7, 2015

Thank You for answer @am11. I've tested node-sass@3.0.0-beta.5 and amazingly simple: node-sass --watch --source-map true scss/custom.scss -o .tmp/css works correctly. Amazingly even the flag --watch works fine! (except every second *.scss save node-sass yields:

{
  "message": "File to read not found or unreadable: D:\\somePath\\scss\\custom.scss",
  "status": 4
}

Despite that source maps works fine :-)

@am11
Copy link
Contributor

am11 commented Apr 8, 2015

except every second ...

This looks like resource acquisition issue with simultaneous I/O operations.
Seems like #801 is related, so lets tackle this issue there.

@am11 am11 closed this as completed Apr 8, 2015
jiongle1 pushed a commit to scantist-ossops-m2/node-sass that referenced this issue Apr 7, 2024
Remove the non-standard image-url function
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Projects
None yet
Development

No branches or pull requests

2 participants