Skip to content

Configurable error visibility options and gutter errors tooltip #726

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

Merged
merged 2 commits into from
Sep 11, 2019

Conversation

bertilnilsson
Copy link
Contributor

Implements two additional configuration options, one for setting an optional gutter icon on lines with errors and one for switching from default squiggly line to solid outline. This is superset of PR #625 functionality.

Also adds a gutter tooltip to show all errors on a line.

Fixes #522 #711

Thoughts and feedback welcome.

… more visible outlining of errors

Also adds a gutter tooltip to show all errors on a line
@msftclas
Copy link

msftclas commented Aug 31, 2019

CLA assistant check
All CLA requirements met.

Copy link
Contributor

@orta orta left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me

Fix typo

Co-Authored-By: Orta <[email protected]>
@orta
Copy link
Contributor

orta commented Sep 11, 2019

👍

@orta orta merged commit 775cbc0 into microsoft:master Sep 11, 2019
@betorobson
Copy link

I have tried this configs but did not work!!!!
Screenshot from 2019-09-16 16-48-45

@bertilnilsson
Copy link
Contributor Author

@betorobson What exactly is not working? Are you getting an error? Is it just the config options that don't do anything? Obviously assume you're actually running this code.

@betorobson
Copy link

betorobson commented Sep 18, 2019

I would like to use a outlined error lint instead the squiggly, also I would like to see a circle in gutter but the only property that works is the color. Look my screenshot, you can see at left panel my source code with squiggly errors and at right panel my Typescript settings.

Screenshot from 2019-09-18 09-59-00

@bertilnilsson
Copy link
Contributor Author

bertilnilsson commented Sep 18, 2019

@betorobson Are you actually running the code from master? This PR has been merged, but a new version has not yet been released.

@betorobson
Copy link

Yes, I assumed that is was working because it is in master branch README.

@bertilnilsson
Copy link
Contributor Author

@betorobson Sorry, I'm still not clear from what you're saying if you're positively running this code or not. If I use your settings with this code, it works as expected.

@betorobson
Copy link

I'm using the last version 3.6.2. As you told me, the settings that i'm trying are only in master branch. Do you have any idea when it will be published?

@@ -5,5 +5,7 @@

// empty string, or one of "region.redish", "region.orangish", "region.yellowish", "region.greenish", "region.bluish", "region.purplish", "region.pinkish"
"error_color": "",
"error_icon": "",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This means there's always a default, which means users haven't been seeing errors since we sent out the update last week.

settings.get("error_color", "invalid.illegal"),
settings.get("error_icon", ""),
sublime.DRAW_OUTLINED
if settings.get("error_outlined") else
sublime.DRAW_NO_FILL +
sublime.DRAW_NO_OUTLINE +
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+ is wrong, so this has never worked...

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.

Show errors in gutter?
5 participants