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

[Feature request] Add an option to display errors inline #27703

Closed
nchevobbe opened this issue May 31, 2017 · 4 comments
Closed

[Feature request] Add an option to display errors inline #27703

nchevobbe opened this issue May 31, 2017 · 4 comments
Assignees
Labels
*duplicate Issue identified as a duplicate of another issue(s)

Comments

@nchevobbe
Copy link

Issue

At the moment, errors are underlined with a wavy red line:

screen shot 2017-05-31 at 09 57 57

And you have 3 ways to see what the error is :

  1. hover the line with the mouse
  2. navigate the cursor to the error and hit ctrl+k and then ctrl+i
  3. hit F8 until you navigate to the errors

Those all work great but need some extra-work

  1. You have to quit the keyboard
  2. you have to navigate to the error, and then remember the 2 key shortcuts
  3. if there are may errors, you might have to loop through a bunch of them before getting the one you're interested in

Of course you also could have the "Errors" panel open, but again, if you have many errors the one you interested in might be buried below what's visible without scrolling.

Request

It would be nice if there was an option to display errors next to the line containing those errors, so they'll show up without any user-interaction.
Here's what it could look like :

screen shot 2017-05-31 at 10 05 13

Challenges

  • The line might be a long one, and the error message would then not be visible
  • In case there are multiple errors on a same line, how do you display it

The errors could be displayed on the next line instead, but from what I saw inspecting the app, it might be tricky to do since the "decorators" are positioned absolutely beneath the actual line of code.

@nchevobbe
Copy link
Author

If this feature request sounds legit, I can try to work on it.

@hoovercj
Copy link
Member

Alternative approach:

Add an option that keeps all errors visible using the current "f8" behavior of showing the errors below the line. The screenshot below shows one error below the line. The configuration option could make it such that these are always visible, and multple errors will display on multiple lines.

image

@nchevobbe
Copy link
Author

@hoovercj this would be great, and easier to implement I guess

@jrieken
Copy link
Member

jrieken commented May 31, 2017

dupe of #19749

@jrieken jrieken closed this as completed May 31, 2017
@jrieken jrieken added the *duplicate Issue identified as a duplicate of another issue(s) label May 31, 2017
@vscodebot vscodebot bot locked and limited conversation to collaborators Nov 18, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
*duplicate Issue identified as a duplicate of another issue(s)
Projects
None yet
Development

No branches or pull requests

3 participants