Skip to content

settings_ui: Fix reset to default not clearing focused input fields - #59395

Merged
MartinYe1234 merged 1 commit into
mainfrom
fix-settings-reset-to-default-button
Jun 16, 2026
Merged

settings_ui: Fix reset to default not clearing focused input fields#59395
MartinYe1234 merged 1 commit into
mainfrom
fix-settings-reset-to-default-button

Conversation

@MartinYe1234

Copy link
Copy Markdown
Contributor

When a setting was reset to its default while its input field was focused, the field kept showing the old value. The reconciliation that syncs the editor with the current setting value now handles cleared (null/empty) defaults and reconciles focused fields when they have no unsaved edits, so the field clears as expected.

Release Notes:

  • Fixed settings input fields not clearing when a setting was reset to its default while the field was focused

@MartinYe1234 MartinYe1234 self-assigned this Jun 16, 2026
@cla-bot cla-bot Bot added the cla-signed The user has signed the Contributor License Agreement label Jun 16, 2026
@zed-community-bot zed-community-bot Bot added the staff Pull requests authored by a current member of Zed staff label Jun 16, 2026
@MartinYe1234
MartinYe1234 marked this pull request as ready for review June 16, 2026 01:57
@MartinYe1234

Copy link
Copy Markdown
Contributor Author

Before:

Screen.Recording.2026-06-15.at.6.58.14.PM.mov

Now

Screen.Recording.2026-06-15.at.7.03.54.PM.mov

@MartinYe1234
MartinYe1234 added this pull request to the merge queue Jun 16, 2026
Merged via the queue into main with commit c270e65 Jun 16, 2026
43 checks passed
@MartinYe1234
MartinYe1234 deleted the fix-settings-reset-to-default-button branch June 16, 2026 03:10
This was referenced Jun 18, 2026
jolutz pushed a commit to jolutz/zed that referenced this pull request Aug 8, 2026
…ed-industries#59395)

When a setting was reset to its default while its input field was
focused, the field kept showing the old value. The reconciliation that
syncs the editor with the current setting value now handles cleared
(null/empty) defaults and reconciles focused fields when they have no
unsaved edits, so the field clears as expected.

Release Notes:

- Fixed settings input fields not clearing when a setting was reset to
its default while the field was focused
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.

2 participants