Skip to content

editor: Fix bracket color change when system theme changes#47505

Merged
SomeoneToIgnore merged 1 commit intozed-industries:mainfrom
smitchaudhary:fix_bracket_colorization
Jan 23, 2026
Merged

editor: Fix bracket color change when system theme changes#47505
SomeoneToIgnore merged 1 commit intozed-industries:mainfrom
smitchaudhary:fix_bracket_colorization

Conversation

@smitchaudhary
Copy link
Contributor

@smitchaudhary smitchaudhary commented Jan 23, 2026

Closes #47503

I added an observer for GlobalTheme changes that check if the accent colors have changed or not. I thought this was closest to the patterns and style in the codebase. I also considered triggering a SettingsStore notification on theme reload but that seemed incorrect since the settings didn't really change, even if it would solve the problem (as the editor already observes SettingsStore to detect theme changes.

Release Notes:

  • Fixed Bracket color not updating when system theme changes

@cla-bot cla-bot bot added the cla-signed The user has signed the Contributor License Agreement label Jan 23, 2026
Copy link
Contributor

@SomeoneToIgnore SomeoneToIgnore left a comment

Choose a reason for hiding this comment

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

Thanks!

@SomeoneToIgnore SomeoneToIgnore enabled auto-merge (squash) January 23, 2026 20:32
@SomeoneToIgnore SomeoneToIgnore merged commit 5d8fd87 into zed-industries:main Jan 23, 2026
30 checks passed
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bracket coloring does not respect theme change like syntax highlighting does

2 participants