Skip to content

Highlight control flow keywords using keyword.control - #66

Merged
MrSubidubi merged 1 commit into
zed-extensions:mainfrom
syormon:main
Sep 29, 2026
Merged

MrSubidubi merged 1 commit into
zed-extensions:mainfrom
syormon:main

Conversation

@syormon

@syormon syormon commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

Its hard to tell at a glance which lines are variables and which are part are logic in Zed.
old_zed

VSCode splits variable keywords from conditional/control flow/function keywords.
vscode

I'm proposing a fix that changes the default theme color for local/global keywords. Here is what it looks like updated in Zed:
new_zed

@cla-bot

cla-bot Bot commented Sep 11, 2026

Copy link
Copy Markdown

We require contributors to sign our Contributor License Agreement, and we don't have @sspivey98 on file. You can sign our CLA at https://zed.dev/cla. Once you've signed, post a comment here that says '@cla-bot check'.

@syormon

syormon commented Sep 11, 2026

Copy link
Copy Markdown
Contributor Author

@cla-bot check

@cla-bot cla-bot Bot added the cla-signed label Sep 11, 2026
@cla-bot

cla-bot Bot commented Sep 11, 2026

Copy link
Copy Markdown

The cla-bot has been summoned, and re-checked this pull request!

@MrSubidubi MrSubidubi changed the title split color theming for local/global from conditionals + control flow Highlightt control flow keywords using keyword.control Sep 29, 2026
@MrSubidubi MrSubidubi changed the title Highlightt control flow keywords using keyword.control Highlight control flow keywords using keyword.control Sep 29, 2026

@MrSubidubi MrSubidubi 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 good, thanks! And congrats to your first contribution! 🎉

@MrSubidubi
MrSubidubi enabled auto-merge (squash) September 29, 2026 12:52
@MrSubidubi
MrSubidubi merged commit c35f93c into zed-extensions:main Sep 29, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants