Skip to content

markdown: Fix inline HTML block highlighting - #61212

Merged
ChristopherBiscardi merged 2 commits into
zed-industries:mainfrom
feitreim:bugfix-inline-markdown
Jul 20, 2026
Merged

markdown: Fix inline HTML block highlighting#61212
ChristopherBiscardi merged 2 commits into
zed-industries:mainfrom
feitreim:bugfix-inline-markdown

Conversation

@feitreim

Copy link
Copy Markdown
Collaborator

Objective

Fixes #59304

Right now inline html tags don't render with any kind of highlighting, despite the fact that non-inline html tags do get highlighting.

Solution

Right now zed already has tree-sitter rules that identifies the inline blocks so we can add a new tree-sitter grammar rule,

((html_tag) @injection.content
  (#set! injection.language "html")
  (#set! injection.combined))

that passes these blocks to the html language server.

Testing

I did some manual testing:

Screen.Recording.2026-07-16.at.3.42.08.PM.mov

And also added a test:
test_markdown_inline_html_highlighting

Self-Review Checklist:

  • I've reviewed my own diff for quality, security, and reliability
  • Unsafe blocks (if any) have justifying comments
  • The content adheres to Zed's UI standards (UX/UI and icon guidelines)
  • Tests cover the new/changed behavior
  • Performance impact has been considered and is acceptable

Release Notes:

  • markdown: Fixed inline html block highlighting.

@cla-bot cla-bot Bot added the cla-signed The user has signed the Contributor License Agreement label Jul 17, 2026
@zed-community-bot zed-community-bot Bot added the guild Pull requests by someone in Zed Guild. NOTE: the label application is automated via github actions label Jul 17, 2026
@danilo-leal danilo-leal changed the title markdown: markdown: Fix inline HTML block highlighting Jul 17, 2026

@ChristopherBiscardi ChristopherBiscardi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

looks great, thanks!

@ChristopherBiscardi
ChristopherBiscardi added this pull request to the merge queue Jul 20, 2026
@ChristopherBiscardi ChristopherBiscardi self-assigned this Jul 20, 2026
Merged via the queue into zed-industries:main with commit 8677759 Jul 20, 2026
51 checks passed
jolutz pushed a commit to jolutz/zed that referenced this pull request Aug 8, 2026
# Objective

Fixes zed-industries#59304

Right now inline html tags don't render with any kind of highlighting,
despite the fact that non-inline html tags do get highlighting.

## Solution

Right now zed already has tree-sitter rules that identifies the inline
blocks so we can add a new tree-sitter grammar rule,
```
((html_tag) @injection.content
  (#set! injection.language "html")
  (#set! injection.combined))
```
that passes these blocks to the html language server.

## Testing

I did some manual testing:


https://github.com/user-attachments/assets/92e5b322-bbc1-48c3-a615-0d32acc2e7d6

And also added a test:
test_markdown_inline_html_highlighting


## Self-Review Checklist:

- [x] I've reviewed my own diff for quality, security, and reliability
- [x] Unsafe blocks (if any) have justifying comments
- [x] The content adheres to Zed's UI standards
([UX/UI](https://github.com/zed-industries/zed/blob/main/CONTRIBUTING.md#uiux-checklist)
and
[icon](https://github.com/zed-industries/zed/blob/main/crates/icons/README.md)
guidelines)
- [x] Tests cover the new/changed behavior
- [x] Performance impact has been considered and is acceptable

---

Release Notes:

- markdown: Fixed inline html block highlighting.
playdohface pushed a commit to playdohface/zed that referenced this pull request Aug 29, 2026
# Objective

Fixes zed-industries#59304

Right now inline html tags don't render with any kind of highlighting,
despite the fact that non-inline html tags do get highlighting.

## Solution

Right now zed already has tree-sitter rules that identifies the inline
blocks so we can add a new tree-sitter grammar rule,
```
((html_tag) @injection.content
  (#set! injection.language "html")
  (#set! injection.combined))
```
that passes these blocks to the html language server.

## Testing

I did some manual testing:


https://github.com/user-attachments/assets/92e5b322-bbc1-48c3-a615-0d32acc2e7d6

And also added a test:
test_markdown_inline_html_highlighting


## Self-Review Checklist:

- [x] I've reviewed my own diff for quality, security, and reliability
- [x] Unsafe blocks (if any) have justifying comments
- [x] The content adheres to Zed's UI standards
([UX/UI](https://github.com/zed-industries/zed/blob/main/CONTRIBUTING.md#uiux-checklist)
and
[icon](https://github.com/zed-industries/zed/blob/main/crates/icons/README.md)
guidelines)
- [x] Tests cover the new/changed behavior
- [x] Performance impact has been considered and is acceptable

---

Release Notes:

- markdown: Fixed inline html block highlighting.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed The user has signed the Contributor License Agreement guild Pull requests by someone in Zed Guild. NOTE: the label application is automated via github actions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Inline HTML comments not highlighted like block comments in Markdown

2 participants