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

lsp.client: mark occurrences sidebar and make occurrences highlighting configurable #7807

Merged
merged 1 commit into from
Oct 18, 2024

Conversation

matthiasblaesing
Copy link
Contributor

Configuration:

image

Result:

image

Important observations:

  1. The usages of the heroes members are not only highlighted in the code, but also on the right sidebar
  2. The cursor is not over heroes anymore, yet the highlight is retained, as no other highlighting is active (this is the result of the "Keep Marks" setting)

@matthiasblaesing matthiasblaesing added LSP [ci] enable Language Server Protocol tests Editor ci:dev-build [ci] produce a dev-build zip artifact (7 days expiration, see link on workflow summary page) labels Sep 27, 2024
@matthiasblaesing matthiasblaesing added this to the NB24 milestone Sep 27, 2024
Copy link
Contributor

@lahodaj lahodaj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, thanks!

@matthiasblaesing
Copy link
Contributor Author

@lahodaj thanks for review. I rebased the change onto master and resolved the merge conflict. Will merge once green.

@matthiasblaesing matthiasblaesing merged commit c7da182 into apache:master Oct 18, 2024
31 checks passed
@matthiasblaesing matthiasblaesing deleted the lsp_mark_provider branch October 18, 2024 17:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci:dev-build [ci] produce a dev-build zip artifact (7 days expiration, see link on workflow summary page) Editor LSP [ci] enable Language Server Protocol tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants