[PR #8835] Bugfix: navigate to new profile ref on settings reload #27311

Closed
opened 2026-01-31 09:21:15 +00:00 by claunia · 0 comments
Owner

Original Pull Request: https://github.com/microsoft/terminal/pull/8835

State: closed
Merged: Yes


Upon a settings reload, we would select the correct navigation item for
a profile, but navigate to the old one. As a result, you would navigate
to the old page that points to a dead profile object. This would make it
appear like you did not discard/save the changes.

This bugfix navigates to the newly created profile, ensuring that your
changes are actually applied to the settings model's clone in use.

References

#8773 - Introduced the bug
#6800 - Settings UI Epic

Validation Steps Performed

  • Navigate to "powershell" profile
  • edit "tab title" value
  • discard changes

Before: changes would persist unless you discarded changes again
Now: changes are discarded

Also verified expected behavior occurs when you click "save" instead of
"discard"

**Original Pull Request:** https://github.com/microsoft/terminal/pull/8835 **State:** closed **Merged:** Yes --- Upon a settings reload, we would select the correct navigation item for a profile, but navigate to the old one. As a result, you would navigate to the old page that points to a dead profile object. This would make it appear like you did not discard/save the changes. This bugfix navigates to the newly created profile, ensuring that your changes are actually applied to the settings model's clone in use. ## References #8773 - Introduced the bug #6800 - Settings UI Epic ## Validation Steps Performed - Navigate to "powershell" profile - edit "tab title" value - discard changes Before: changes would persist unless you discarded changes again Now: changes are discarded Also verified expected behavior occurs when you click "save" instead of "discard"
claunia added the pull-request label 2026-01-31 09:21:15 +00:00
Sign in to join this conversation.
No Label pull-request
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#27311