[PR #3892] Add support for "User Default" settings II #25547

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

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

State: closed
Merged: Yes


Summary of the Pull Request

This is attempt 2 at this feature. The original PR can be found at #3369.

These are settings that apply to every profile, before user customizations.

If the user wants to add "default profile settings", they can make the "profiles" property an object, instead of a list, and add "defaults" key underneath that object. The users list of profiles should then be under the list property of the profiles object.

References

#2515, #2603, #3369, #3569

PR Checklist

Detailed Description of the Pull Request / Additional comments

Discussion in #2325 itself serves as the "spec" for this task. I thought we'd need more discussion on the topic, but it ended up being pretty straightforward.

I should not have said that in the original PR. We've had a better spec review now that I think we're happier with.

Validation Steps Performed

ran the tests

**Original Pull Request:** https://github.com/microsoft/terminal/pull/3892 **State:** closed **Merged:** Yes --- ## Summary of the Pull Request _This is attempt 2 at this feature_. The original PR can be found at #3369. These are settings that apply to _every_ profile, before user customizations. If the user wants to add "default profile settings", they can make the `"profiles"` property an _object_, instead of a list, and add `"defaults"` key underneath that object. The users list of profiles should then be under the `list` property of the `profiles` object. ## References #2515, #2603, #3369, #3569 ## PR Checklist * [x] Closes #2325 * [x] I work here * [x] Tests added/passed * [x] schema, docs updated ## Detailed Description of the Pull Request / Additional comments ~~Discussion in #2325 itself serves as the "spec" for this task. I thought we'd need more discussion on the topic, but it ended up being pretty straightforward.~~ I should not have said that in the original PR. We've had a better spec review now that I think we're happier with. ## Validation Steps Performed _ran the tests_
claunia added the pull-request label 2026-01-31 09:10:14 +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#25547