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

Don't show hint diagnostics in problems view? #45436

Closed
mjbvz opened this issue Mar 9, 2018 · 6 comments
Closed

Don't show hint diagnostics in problems view? #45436

mjbvz opened this issue Mar 9, 2018 · 6 comments
Assignees
Labels
*duplicate Issue identified as a duplicate of another issue(s) error-list Problems view feature-request Request for new features or functionality languages-diagnostics Source problems reporting release-notes Release notes issues under-discussion Issue is under discussion for relevance, priority, approach
Milestone

Comments

@mjbvz
Copy link
Collaborator

mjbvz commented Mar 9, 2018

Problem
TypeScript is introducing new "suggest" diagnostics (#45378 + #45433) that seem to map to our DiagnosticServerity.Hint level. Example suggest diagnostic: convert a es5 class to an es6 class.

However these hint diagnostics still are still show in the problems view, which adds a lot of noise

screen shot 2018-03-09 at 12 13 16 pm

Proposal
Don't show hint diagnostics in the problems panel. They are already not shown in the editor. Not sure if any extensions depend on the current functionality

@mjbvz mjbvz added this to the March 2018 milestone Mar 9, 2018
@mjbvz
Copy link
Collaborator Author

mjbvz commented Mar 9, 2018

If we are concerned about regressing extensions, two other options:

  • Introduce a new, even lower severity diagnostic level that never shows in the UI

  • Allow diagnostics or diagnostics collection themselves to say if they show in the UI or not

@sandy081
Copy link
Member

@jrieken Since, we are planning to move F8 functionality being driven from editor to problems view, does it mean that F8 now will also navigate through all types of diagnostics?

How about adding a setting to exclude diagnostics by type which can be language overridden?

@sandy081 sandy081 added feature-request Request for new features or functionality under-discussion Issue is under discussion for relevance, priority, approach labels Mar 12, 2018
@jrieken
Copy link
Member

jrieken commented Mar 13, 2018

We should always ignore Hint (the editor already does that today), the F8 likely doesn't... We should add something to IMarkerService#read so that it filters on severities

@mjbvz
Copy link
Collaborator Author

mjbvz commented Mar 13, 2018

We can discuss this at the UX sync but I think a good experience would be:

  • By default, TS suggestion diagnostics such as convert a es5 class to an es6 class show as faint gray underlines in the editor. This would let users know where they can perform code actions. These diagnostics would not show in the problems view or participate in F8

  • We provide an option this hide these suggestion diagnostics in the editor as well. TS is looking into per-error suppressions but I think we need global solution on the VS Code side as well

Not sure how these would map to our diagnostic api

@sandy081 sandy081 added error-list Problems view languages-diagnostics Source problems reporting labels Mar 14, 2018
@jrieken
Copy link
Member

jrieken commented Mar 14, 2018

This is somewhat the same as #44141

@jrieken jrieken added the *duplicate Issue identified as a duplicate of another issue(s) label Mar 14, 2018
@vscodebot
Copy link

vscodebot bot commented Mar 14, 2018

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for existing issues here. See also our issue reporting guidelines.

Happy Coding!

@vscodebot vscodebot bot closed this as completed Mar 14, 2018
@jrieken jrieken added the release-notes Release notes issues label Mar 28, 2018
@vscodebot vscodebot bot locked and limited conversation to collaborators Apr 28, 2018
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) error-list Problems view feature-request Request for new features or functionality languages-diagnostics Source problems reporting release-notes Release notes issues under-discussion Issue is under discussion for relevance, priority, approach
Projects
None yet
Development

No branches or pull requests

3 participants