LSP icon indicating copy to clipboard operation
LSP copied to clipboard

Diagnostics panel user setting ignored

Open lapega opened this issue 3 years ago • 6 comments

Ubuntu 20.04 - ST4

I have this setting in LSP.sublime-settings

{
	"show_diagnostics_panel_on_save": 1,
}

But the panel also opens on warnings, so now I have to set to 0 to keep it closed, but would be nice to have this setting working.

Thanks!

lapega avatar May 28 '21 13:05 lapega

I cannot reproduce this.

https://user-images.githubusercontent.com/2431823/120047614-505c3b00-c015-11eb-89be-e577537f3b31.mov

In the video above you can see me saving a file as can be seen from the status bar. I then open the diagnostics panel to reveal one warning present. I have show_diagnostics_panel_on_save set to 1 in my settings.

What langserver are you using?

rwols avatar May 28 '21 22:05 rwols

I'm using php, the problem doesn't happens all the time, I can't reproduce the behaviour, but seems to be randomly. (the panel opens when are hints to show)

imagen

lapega avatar May 31 '21 12:05 lapega

I think the error triggered the diagnostic panel, but in a flash the error disappeared and the panel didn't close automatically. I've had that happen

husanjun avatar Jun 02 '21 03:06 husanjun

I think the error triggered the diagnostic panel, but in a flash the error disappeared and the panel didn't close automatically. I've had that happen

There is no auto-close functionality so that would be expected (and not related to this issue, I believe).

rchl avatar Jun 02 '21 06:06 rchl

There is no auto-close functionality so that would be expected (and not related to this issue, I believe).

Thanks, Let me explain that when there is an error in the view (which may be caused by some reason), we save the view before the diagnostic response arrives, which will trigger the panel, and then the response arrives it is good

husanjun avatar Jun 02 '21 10:06 husanjun

Thanks, Let me explain that when there is an error in the view (which may be caused by some reason), we save the view before the diagnostic response arrives, which will trigger the panel, and then the response arrives it is good

I think you're correct @husanjun. There's not much we can do about that.

rwols avatar Jun 04 '21 18:06 rwols

Hopefully this is addressed with https://github.com/sublimelsp/LSP/pull/2063 I tried to reproduce it, but could not.

predragnikolic avatar Sep 26 '22 13:09 predragnikolic

It should be addressed, yes.

rchl avatar Sep 26 '22 14:09 rchl