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

plugin: support 'deprecated' strikethrough #8553

Merged
merged 1 commit into from
Oct 5, 2020
Merged

Conversation

vince-fugnitto
Copy link
Member

What it does

Fixes: #8552

The following pull-request adds support for the following:

  • support for tags strikethrough for deprecated completion items.
  • support for the deprecated deprecated flag for completion items.

Both the tag and deprecated properties should be visibly different (strikethrough) to indicate deprecated items.

deprecated

How to test

  1. include the following extesion: lsp-sample-deprecated
  2. start the application
  3. create a sample textfile
  4. trigger auto-suggestion/completion
  5. the deprecated items should be striked-through

Review checklist

Reminder for reviewers

Signed-off-by: vince-fugnitto vincent.fugnitto@ericsson.com

@vince-fugnitto vince-fugnitto added editor issues related to the editor vscode issues related to VSCode compatibility labels Sep 23, 2020
@vince-fugnitto vince-fugnitto self-assigned this Sep 23, 2020
@DucNgn
Copy link
Contributor

DucNgn commented Sep 24, 2020

I tested and confirmed the PR works exactly as expected 👍

The following pull-request adds support for the following:
- support for `tags` strikethrough for deprecated completion items.
- support for the deprecated `deprecated` flag for completion items.

Signed-off-by: vince-fugnitto <vincent.fugnitto@ericsson.com>
Copy link
Member

@amiramw amiramw left a comment

Choose a reason for hiding this comment

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

Code LGTM. Also tested and the fix work.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
editor issues related to the editor vscode issues related to VSCode compatibility
Projects
None yet
Development

Successfully merging this pull request may close these issues.

No indication for deprecated variables
3 participants