-
Notifications
You must be signed in to change notification settings - Fork 30.2k
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
Allow control over what extensions contribute to Problems list (and File Explorer) #44141
Comments
I like.. Two option that I can think of: (1) we have a setting which spells out the extensions and the places in which they can show errors/warning or (2) we add a new property the diagnostics api to signal that a diagnostics is 'local', e.g only to be shown in the editor... I kind of prefer the latter but it needs buy-in from extension authors. |
I think both sounds like useful features; (2) Is good to allow extension developers to provide sensible defaults. |
Idea from #45436 is to show diagnostics with severity Hint only in the editor... |
So that would require the extension to update their diagnostics type to this new proposed type? |
The type isn't new, there already existed Error, Warning, Information, and Hint. Most extensions I have checked are already using Hint or Information, e.g. spell checkers, so it should be relatively smooth. First step will be not show Hint in the error panel nor the explorer. |
|
Instead of not showing it completely, how about providing an option (check box) for showing types of problems. By default hints are deselected? |
Hm, unsure.... for information yeah but for hints I am unsure |
re #44141 (comment) @DustinCampbell @stevencl how does VS show diagnostics with severity hint? Grey underline as @mjbvz suggests? |
Excluded |
ping @stevencl |
Thanks |
@stevencl How does this look like for a large (real-life) piece of code? Is there an upper bound to the number of suggestions displayed? Is this the default experience in Visual Studio? I am concerned that these suggestions might end up dominating the source code or interfering significantly with the source code coloring / presentation. |
@mjbvz How will TypeScript use this? Will the whole document be full of severity hint diagnostics (basically for each and every applicable code action) or will TypeScript only use it for must-do code actions? |
Note that in VS, there are 33 general style rules for C# that you can configure the editor to provide suggestions for. Of these, only 16 are set to provide a suggestion by default. The others aren't turned on. |
Cool! cc @rchande |
@mjbvz see #44141 (comment), you might wanna add some config-settings for TypeScript/JavaScript suggestions |
Will there be further options to control what extensions contribute to the Problems/Explorer panel? Asking because of this comment: streetsidesoftware/vscode-spell-checker#186 (comment) The poster used an insider version with the changes that hides Hints;
That worked, but misspelled words then only have the |
I'm using a spell checker extension with VSCode with the intent of seeing the squiggly underlines in the editor for spelling issues.
However, these also get summarised under the Problems tab, and more recently they contribute to the error notation in the Folder view.
These warnings quickly gets noisy. I wish I could control which extensions contribute to these issue
summaries. For spell checking I wish to see the squiggly underline in the editor, but not see them summarized elsewhere.
From what I can tell, my current options are either to disable the spell checker (not a solution) or disable the Folder view warnings (also not desirable as I want warnings from other sources.).
I originally filed a feature request with the extension itself: streetsidesoftware/vscode-spell-checker#186 (comment)
But I was told that it wasn't under the control of the extension and rather a feature of VSCode itself.
Is there any chance this could be made possible? Maybe a setting once can configure to disable certain providers for the Problems panel and Folder view warnings?
The text was updated successfully, but these errors were encountered: