Skip to content

languages: Reorder type identifier highlights in JavaScript#49325

Merged
yeskunall merged 2 commits intomainfrom
fix-issue-49226-6b4f4d1c5b938caf
Feb 27, 2026
Merged

languages: Reorder type identifier highlights in JavaScript#49325
yeskunall merged 2 commits intomainfrom
fix-issue-49226-6b4f4d1c5b938caf

Conversation

@yeskunall
Copy link
Copy Markdown
Member

Move general type identifier rules before class-specific ones to ensure proper precedence in the syntax highlighting query.

Closes #49226.

Before you mark this PR as ready for review, make sure that you have:

  • Added a solid test coverage and/or screenshots from doing manual testing
  • Done a self-review taking into account security and performance aspects
  • Aligned any UI changes with the UI checklist

Release Notes:

  • Fixed an issue where class names were not highlighted correctly in JavaScript files

Move general type identifier rules before class-specific ones to
ensure proper precedence in the syntax highlighting query.
@cla-bot cla-bot bot added the cla-signed The user has signed the Contributor License Agreement label Feb 17, 2026
@zed-community-bot zed-community-bot bot added the staff Pull requests authored by a current member of Zed staff label Feb 17, 2026
@yeskunall yeskunall marked this pull request as draft February 17, 2026 04:54
@yeskunall yeskunall marked this pull request as ready for review February 19, 2026 17:09
@yeskunall yeskunall enabled auto-merge (squash) February 27, 2026 03:09
@yeskunall yeskunall merged commit 1c39e19 into main Feb 27, 2026
28 checks passed
@yeskunall yeskunall deleted the fix-issue-49226-6b4f4d1c5b938caf branch February 27, 2026 03:14
rtfeldman pushed a commit that referenced this pull request Feb 27, 2026
Move general type identifier rules before class-specific ones to ensure
proper precedence in the syntax highlighting query.

Closes #49226.

Before you mark this PR as ready for review, make sure that you have:
- [ ] Added a solid test coverage and/or screenshots from doing manual
testing
- [x] Done a self-review taking into account security and performance
aspects
- [x] Aligned any UI changes with the [UI
checklist](https://github.com/zed-industries/zed/blob/main/CONTRIBUTING.md#uiux-checklist)

Release Notes:

- Fixed an issue where class names were not highlighted correctly in
JavaScript files
tahayvr pushed a commit to tahayvr/zed that referenced this pull request Mar 4, 2026
…stries#49325)

Move general type identifier rules before class-specific ones to ensure
proper precedence in the syntax highlighting query.

Closes zed-industries#49226.

Before you mark this PR as ready for review, make sure that you have:
- [ ] Added a solid test coverage and/or screenshots from doing manual
testing
- [x] Done a self-review taking into account security and performance
aspects
- [x] Aligned any UI changes with the [UI
checklist](https://github.com/zed-industries/zed/blob/main/CONTRIBUTING.md#uiux-checklist)

Release Notes:

- Fixed an issue where class names were not highlighted correctly in
JavaScript files
naaiyy pushed a commit to Glass-HQ/Glass that referenced this pull request Mar 10, 2026
…stries#49325)

Move general type identifier rules before class-specific ones to ensure
proper precedence in the syntax highlighting query.

Closes zed-industries#49226.

Before you mark this PR as ready for review, make sure that you have:
- [ ] Added a solid test coverage and/or screenshots from doing manual
testing
- [x] Done a self-review taking into account security and performance
aspects
- [x] Aligned any UI changes with the [UI
checklist](https://github.com/zed-industries/zed/blob/main/CONTRIBUTING.md#uiux-checklist)

Release Notes:

- Fixed an issue where class names were not highlighted correctly in
JavaScript files
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 staff Pull requests authored by a current member of Zed staff

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Theme token type.class doesn't highlight class names in JS (while works well in TS and TSX)

1 participant