redhat-developer / lsp4ij

LSP Client for IntelliJ
Eclipse Public License 2.0
91 stars 17 forks source link

feat: Support for `textDocument/semanticTokens` #392

Closed angelozerr closed 3 months ago

angelozerr commented 3 months ago

feat: Support for textDocument/semanticTokens

Fixes #238

@FalsePattern, @CppCXY here my current work for semantic tokens support which starts working.

image

   public @Nullable TextAttributesKey getTextAttributesKey(@NotNull String tokenType,
   @NotNull List<String> tokenModifiers,
   @NotNull PsiFile file);

Please read LSPSupport/md documentation that I'm updating (not finished) to see more information like mappings between semantic token/modifiers and the TextAttributeKey.

angelozerr commented 3 months ago

@FalsePattern @CppCXY I have tested my semantic tokens support with several language servers and the result seems good, but the support must be improved again.

As it is working, do you want me to merge this PR ASAP so you can install a nightly build and you can play with semantic tokens, so you can provide feedback?

If you are OK with this idea, I could merge it ASAP, but please consider that this support is experimental and it requires some improvements.

CppCXY commented 3 months ago

As it is working, do you want I merge this PR ASAP in order to that you can install nighly build and you play with semantic tokens to give is feedback to improve it?

I agree, I will also test this feature ASAP

angelozerr commented 3 months ago

I agree, I will also test this feature ASAP

Ok I will try to merge it today.