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

Tooltip: Show full selector path. (#371) #828

Merged
merged 1 commit into from
Mar 31, 2014

Conversation

am11
Copy link
Contributor

@am11 am11 commented Mar 31, 2014

Fixes #371. Works fine for LESS.

For SCSS, even the hacky solution doesn't guarantee if all selectors would be mapped to the source (eventually we need to match the selector name which bring us back to square 1: the reason we chose source-map over our manual LessDocument/ScssDocument approach).

FWIW, its the issues with libsass, (the backbone of node-sass, which we are using). However, the original sass gem generates correct source maps. See sass/libsass#324 (comment).

@madskristensen, this PR is isolated from the previous one (its coming from separate branch). I have made changes in (two) separate files. It can be merged for v2 without breaking anything. 🏁

madskristensen added a commit that referenced this pull request Mar 31, 2014
Tooltip: Show full selector path. (#371)
@madskristensen madskristensen merged commit 646d275 into madskristensen:master Mar 31, 2014
@am11 am11 deleted the css-selector-tooltip branch March 31, 2014 17:00
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.

Full selector tooltip no showing on nested LESS selectors
2 participants