-
-
Notifications
You must be signed in to change notification settings - Fork 0
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
Debug links #76
Debug links #76
Conversation
✅ Deploy Preview for sass-site-oddbird ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
@mirisuzanne or @stacyk This is ready for design. |
@jamesnw I added another example, which has all three, and will scroll you down the page a bit. I also wanted one that tests 3 digit gutters here. I also tested it with 4 and 5 and it works, but it is pretty tight at 5. I kept the bullet since I couldn't change the line number associated as there was not semantic link to the line that has a notice in the markup. I attempted to do a bg color pseduo element, but since I didn't know how wide to make it as that column is allowed to grow, I decided this bullet works OK with a few tweaks. This is ready for review. |
Moved to sass#1179 |
Description
In the debug console, line numbers are now links to select or position the cursor at the relevant issue. Also, deprecation messages with links are now clickable.
Related Issue(s)
Addresses #75
Steps to test/reproduce
2. Error link
Links should allow you to open in new tab/window with CMD/Shift/Right click, etc.