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

fix: hightlighting breaks after single line doc #92

Conversation

pdm-jd
Copy link
Contributor

@pdm-jd pdm-jd commented Jan 8, 2025

Doc comments like @doc "foo" break syntax highlighting after 0.0.19

#91

Before

VSCode 1.96.2 and Lexical 0.0.19

image

Cursor 0.44.11 and Lexical 0.0.19

image

After

Cursor 0.44.11

image

…ment instead of only the begin and end captures
Copy link
Collaborator

@Blond11516 Blond11516 left a comment

Choose a reason for hiding this comment

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

Thanks for the fix!

I would've liked to maintain markdown highlighting within the string but couldn't get that to work without losing comment coloring on the closing " so I'll approve as-is. I might come back and try again later.

I've also put the comment scope back on the whole line. Since markdown highlighting no longer works there's no point treating the separate parts distinctly.

@Blond11516 Blond11516 enabled auto-merge (squash) January 10, 2025 02:46
@Blond11516 Blond11516 merged commit 77ed588 into lexical-lsp:main Jan 10, 2025
6 checks passed
@pdm-jd pdm-jd deleted the fix/highlighting-breaks-after-single-line-doc branch January 10, 2025 18:18
@pdm-jd
Copy link
Contributor Author

pdm-jd commented Jan 10, 2025

Great, thanks for getting to this so quickly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants