Hi @dbaeumer while working on this PR I noticed that the status bar color is not changing on error or warning status, it's on purpose? Maybe with the option to silent server errors we should highlight the status bar to show to the user that something is wrong and the file is not being parsed, what do you think?

Another option that I thought about adding with this (or another) PR is some commands to allow the user to silent or display all server errors for the session. Could this be a good option when the error message changes on repeated errors?

Maybe this could be helpful to alleviate other issues like #1238

@edupsousa sorry for the delay :-)

We should definitely update the color. It did in the past, but the code got somehow lost :-). It is here:

The background is never updated.

I would actually not add any additional commands. The reasons are:

• I think not showing the same message over and over is a good default
• clicking on the ESLint status takes you to the output channel.

@edupsousa sorry for the delay :-)

We should definitely update the color. It did in the past, but the code got somehow lost :-). It is here:

vscode-eslint/client/src/extension.ts

Line 834 in 7568b4c

let backgroundColor: ThemeColor | undefined;

The background is never updated.

I would actually not add any additional commands. The reasons are:

• I think not showing the same message over and over is a good default
• clicking on the ESLint status takes you to the output channel.

No problem with the delay. The year is ending but the work just increases. :-)

I was playing around another day and already did the background color update, but the branch where I did it is polluted with other things that I was playing. I will do some cleanup and tests before submitting it in a new PR.

@edupsousa I looked at the code again and did you do any chances to update the background color? It is not in the PR.

@edupsousa I looked at the code again and did you do any chances to update the background color? It is not in the PR.

@dbaeumer, I was planning to put it on a separate PR, sorry for the confusion. But, to keep it simple I just added the status bar color code to this same PR.

@edupsousa thanks. Will have a look