Skip to content

Conversation

@subyssurendran666
Copy link
Contributor

This PR fixes an issue where the {@code} inline tag is not rendered correctly in the Markdown hover. It also addresses incorrect handling of the {@literal} tag, which is included as part of the same issue.
In {@literal} tags, below characters are escaped and rendered accordingly.

<>*^&\[]`

Fix: #3672

This PR fixes an issue where the {@code} inline tag is not rendered
correctly in the Markdown hover. It also addresses incorrect handling of
the {@literal} tag, which is included as part of the same issue.

Fix: eclipse-jdtls#3672
Copy link
Contributor

@fbricon fbricon left a comment

Choose a reason for hiding this comment

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

Multiline {@code } breaks, e.g.

/// Here's some code {@code 
///     List<String> list = List.of("Hello World!");
/// } 
/// that does something.
Screenshot 2026-01-19 at 12 55 36

While the same in HTML is rendered properly:

/**
 * Here's some code {@code 
 *  List<String> list = List.of("Hello World!");
 * }
 * that does something.
 */
Screenshot 2026-01-19 at 12 55 03

@subyssurendran666 subyssurendran666 force-pushed the code_tag_incorrectly_rendered_in_markdown_3672 branch from b3bd591 to 2440bbb Compare January 20, 2026 07:47
@subyssurendran666
Copy link
Contributor Author

Thanks for reviewing this PR and for the feedback. I have pushed the latest changes. The third and fourth unit tests (testHoverMarkdownWithCodeTag_03 and testHoverMarkdownWithCodeTag_04) will always fail, since there is a bug in JDT.Core. The above-mentioned tests will succeed once the corresponding PR gets merged.

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.

{@code} inline tag is not rendered correctly in Markdown hover

2 participants