[PR #11447] Fix layering issues with CascadiaSettings::_createNewProfile #28591

Open
opened 2026-01-31 09:29:29 +00:00 by claunia · 0 comments
Owner

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

State: closed
Merged: Yes


CascadiaSettings::_createNewProfile failed to call _FinalizeInheritance.
This commits fixes the issue and adds a stern warning for future me.

PR Checklist

Validation Steps Performed

  • Open settings UI
  • Modify font size in base layer
  • Don't save
  • Duplicate any profile with default font size
  • Ensure the duplicated profile shows the modified base layer font size ✔️
**Original Pull Request:** https://github.com/microsoft/terminal/pull/11447 **State:** closed **Merged:** Yes --- `CascadiaSettings::_createNewProfile` failed to call `_FinalizeInheritance`. This commits fixes the issue and adds a stern warning for future me. ## PR Checklist * [x] Closes #11392 * [x] I work here * [x] Tests added/passed ## Validation Steps Performed * Open settings UI * Modify font size in base layer * _Don't_ save * Duplicate any profile with default font size * Ensure the duplicated profile shows the modified base layer font size ✔️
claunia added the pull-request label 2026-01-31 09:29:29 +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#28591