[PR #9655] [MERGED] Remove Base Layer from Settings UI #27682

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

📋 Pull Request Information

Original PR: https://github.com/microsoft/terminal/pull/9655
Author: @carlos-zamora
Created: 3/29/2021
Status: ✅ Merged
Merged: 3/30/2021
Merged by: @undefined

Base: main ← Head: dev/cazamor/sui/remove-base-layer


📝 Commits (2)

  • 5a55aa9 Remove Base Layer from Settings UI
  • afa1d96 Merge branch 'main' into dev/cazamor/sui/remove-base-layer

📊 Changes

3 files changed (+5 additions, -21 deletions)

View changed files

📝 src/cascadia/TerminalSettingsEditor/MainPage.cpp (+0 -12)
📝 src/cascadia/TerminalSettingsEditor/MainPage.xaml (+0 -8)
📝 src/cascadia/TerminalSettingsEditor/SettingContainer.cpp (+5 -1)

📄 Description

Removes base layer (aka profiles.defaults) from the Settings UI. SettingContainer was also updated to not present a revert arrow when overriding a base layer value.

The new experience is now as follows:

  • The revert arrow will only appear if you are overriding a value from a fragment extension.
  • Users are still able to fully interact with profiles.defaults in their settings.json. Doing so still propagates those changes to their profiles as normal. In this case, the Settings UI presents the base layer value as the one that you selected.

#6800 - Settings UI Epic
Closes #9539


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/microsoft/terminal/pull/9655 **Author:** [@carlos-zamora](https://github.com/carlos-zamora) **Created:** 3/29/2021 **Status:** ✅ Merged **Merged:** 3/30/2021 **Merged by:** [@undefined](undefined) **Base:** `main` ← **Head:** `dev/cazamor/sui/remove-base-layer` --- ### 📝 Commits (2) - [`5a55aa9`](https://github.com/microsoft/terminal/commit/5a55aa93721573e88cc5d42002ec6f5b648c2c80) Remove Base Layer from Settings UI - [`afa1d96`](https://github.com/microsoft/terminal/commit/afa1d96a9c18a5b3ce735f4a5a3cad2e035dbe2d) Merge branch 'main' into dev/cazamor/sui/remove-base-layer ### 📊 Changes **3 files changed** (+5 additions, -21 deletions) <details> <summary>View changed files</summary> 📝 `src/cascadia/TerminalSettingsEditor/MainPage.cpp` (+0 -12) 📝 `src/cascadia/TerminalSettingsEditor/MainPage.xaml` (+0 -8) 📝 `src/cascadia/TerminalSettingsEditor/SettingContainer.cpp` (+5 -1) </details> ### 📄 Description Removes base layer (aka profiles.defaults) from the Settings UI. `SettingContainer` was also updated to not present a revert arrow when overriding a base layer value. The new experience is now as follows: - The revert arrow will only appear if you are overriding a value from a fragment extension. - Users are still able to fully interact with `profiles.defaults` in their settings.json. Doing so still propagates those changes to their profiles as normal. In this case, the Settings UI presents the base layer value as the one that you selected. #6800 - Settings UI Epic Closes #9539 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
claunia added the pull-request label 2026-01-31 09:23:27 +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#27682