Skip to content
This repository has been archived by the owner on Oct 13, 2024. It is now read-only.

Improved Hover Experience #30

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Improved Hover Experience #30

wants to merge 2 commits into from

Conversation

EnochMtzR
Copy link

The issue:

before_optimized

  • The default hover capability's contentFormat was being sent as plaintext by default.
  • the markdown was not processed correctly, which made &nbsp and escape characters (\) be printed as strings.
  • Improved the documentation to suggest the markdown_inline tree-sitter parser as recommended for a great hover experience

After:

after_optimized

@elan-hwang
Copy link

Is there anything that blocks this PR?

@EnochMtzR
Copy link
Author

@elan-hwang not that I'm aware of!

@iamim
Copy link

iamim commented Jul 2, 2024

Hi @EnochMtzR
Would you mind re-implementing it in https://github.com/seblj/roslyn.nvim?

@tuliopaim tuliopaim mentioned this pull request Aug 22, 2024
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@EnochMtzR @elan-hwang @iamim