[PR #5265] [MERGED] Rename 'requestedTheme' to 'theme' #26209

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

📋 Pull Request Information

Original PR: https://github.com/microsoft/terminal/pull/5265
Author: @carlos-zamora
Created: 4/7/2020
Status: Merged
Merged: 4/7/2020
Merged by: @undefined

Base: masterHead: dev/cazamor/settings/theme


📝 Commits (2)

  • 6f5f334 rename 'requestedTheme' to 'theme'
  • 9eb36cb forgot about universal defaults

📊 Changes

7 files changed (+21 additions, -23 deletions)

View changed files

📝 doc/cascadia/SettingsSchema.md (+1 -1)
📝 doc/cascadia/profiles.schema.json (+1 -1)
📝 src/cascadia/TerminalApp/AppLogic.cpp (+4 -4)
📝 src/cascadia/TerminalApp/GlobalAppSettings.cpp (+9 -9)
📝 src/cascadia/TerminalApp/GlobalAppSettings.h (+4 -6)
📝 src/cascadia/TerminalApp/defaults-universal.json (+1 -1)
📝 src/cascadia/TerminalApp/defaults.json (+1 -1)

📄 Description

Summary of the Pull Request

Renames the requestedTheme global setting to theme. Propagates updates to...

  • schema
  • doc
  • defaults.json
  • universal-defaults.json

PR Checklist

Validation Steps Performed

theme Success?
system
light
dark

But we really know that dark is the one we care about here 😉


🔄 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/5265 **Author:** [@carlos-zamora](https://github.com/carlos-zamora) **Created:** 4/7/2020 **Status:** ✅ Merged **Merged:** 4/7/2020 **Merged by:** [@undefined](undefined) **Base:** `master` ← **Head:** `dev/cazamor/settings/theme` --- ### 📝 Commits (2) - [`6f5f334`](https://github.com/microsoft/terminal/commit/6f5f334a5cac9f8e9fe8ea042eb03002feda45fe) rename 'requestedTheme' to 'theme' - [`9eb36cb`](https://github.com/microsoft/terminal/commit/9eb36cbec2c82d408e30a823985920f61646f604) forgot about universal defaults ### 📊 Changes **7 files changed** (+21 additions, -23 deletions) <details> <summary>View changed files</summary> 📝 `doc/cascadia/SettingsSchema.md` (+1 -1) 📝 `doc/cascadia/profiles.schema.json` (+1 -1) 📝 `src/cascadia/TerminalApp/AppLogic.cpp` (+4 -4) 📝 `src/cascadia/TerminalApp/GlobalAppSettings.cpp` (+9 -9) 📝 `src/cascadia/TerminalApp/GlobalAppSettings.h` (+4 -6) 📝 `src/cascadia/TerminalApp/defaults-universal.json` (+1 -1) 📝 `src/cascadia/TerminalApp/defaults.json` (+1 -1) </details> ### 📄 Description ## Summary of the Pull Request Renames the `requestedTheme` global setting to `theme`. Propagates updates to... - schema - doc - defaults.json - universal-defaults.json ## PR Checklist * [X] Closes #5264 ## Validation Steps Performed | `theme` | Success? | |--|--| | `system` | ✔ | | `light` | ✔ | | `dark` | ✔ | But we really know that `dark` is the one we care about here 😉 --- <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:14:38 +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#26209