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

Diagnostics panel user setting ignored #1695

Closed
lapega opened this issue May 28, 2021 · 8 comments
Closed

Diagnostics panel user setting ignored #1695

lapega opened this issue May 28, 2021 · 8 comments

Comments

@lapega
Copy link

lapega commented May 28, 2021

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!

@rwols
Copy link
Member

rwols commented May 28, 2021

I cannot reproduce this.

Schermopname.2021-05-29.om.00.31.02.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?

@lapega
Copy link
Author

lapega commented May 31, 2021

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

@husanjun
Copy link
Contributor

husanjun commented Jun 2, 2021

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

@rchl
Copy link
Member

rchl commented Jun 2, 2021

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).

@husanjun
Copy link
Contributor

husanjun commented Jun 2, 2021

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

@rwols
Copy link
Member

rwols commented Jun 4, 2021

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.

@predragnikolic
Copy link
Member

Hopefully this is addressed with #2063
I tried to reproduce it, but could not.

@rchl
Copy link
Member

rchl commented Sep 26, 2022

It should be addressed, yes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants