-
Notifications
You must be signed in to change notification settings - Fork 9
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
documentHighlightProvider
capability
#24
Comments
If you open the LSP log panel with “LSP: Toggle Log Panel”, then you can see what capabilities the server has exactly by looking at the response of the |
Maybe easier: run |
@rwols I can't feel sad now that the answer is no since I've also learned something, thanks :) Can we consider this "issue" a feature request then? |
It seems JoeRobich added it just 18 days ago: https://github.com/OmniSharp/omnisharp-roslyn/blob/c214d900ca250556bb09156fcb2b27d217a3a523/src/OmniSharp.LanguageServerProtocol/Handlers/OmniSharpDocumentHighlightHandler.cs |
But maybe it's just a placeholder, I haven't looked much into it: OmniSharp/omnisharp-roslyn#2463 |
v1.39.2 should contain that functionality: https://github.com/OmniSharp/omnisharp-roslyn/releases/tag/v1.39.2 |
While we use v1.38.2: Line 15 in 1afed47
|
I can confirm that replacing the packaged omnisharp version with the latest release 1.39.2 does provide highlighting. |
Should be fixed by PR #40, which makes transition to latest .NET 6+ based OmniSharp binaries. |
I find it useful when other occurances of the symbol under the caret are highlighted, makes it easy seeing the immediate usages. But this doesn't seem to work in LSP-OmniSharp even when I change
document_highlight_style
tobackground
and add the related scopes with custom background colors to my color theme (using this as a guide https://lsp.sublimetext.io/customization/#document-highlights).Do we have the required capability in the server, am I doing something wrong?
The text was updated successfully, but these errors were encountered: