[PR #6993] Add support for changing the active color scheme with an action #26833

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

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

State: closed
Merged: Yes


Summary of the Pull Request

Adds the setColorScheme action, to change the color scheme of the active control to one given by the name parameter. name is required. If name is not the name of a color scheme, the action does nothing.

References

  • Being done as a stepping stone to #6689

PR Checklist

Detailed Description of the Pull Request / Additional comments

Technically, the action is being done by changing the settings of the current TerminalSettings of the TermControl. Frankly, it should be operating on a copy of the TermControl's IControlSettings, then updating the control's settings, or the Control should just listen for changes to it's setting's properties, and update in real time (without a manual call to UpdateSettings. However, both those paths are somewhere unknowable beyond #6904, so we'll just do this for now.

Validation Steps Performed

  • tested manually with a scheme that exists
  • tested manually with a scheme that doesn't exist
**Original Pull Request:** https://github.com/microsoft/terminal/pull/6993 **State:** closed **Merged:** Yes --- ## Summary of the Pull Request Adds the `setColorScheme` action, to change the color scheme of the active control to one given by the `name` parameter. `name` is required. If `name` is not the name of a color scheme, the action does nothing. ## References * Being done as a stepping stone to #6689 ## PR Checklist * [x] Closes #5401 * [x] I work here * [ ] Tests added/passed * [n/a] Requires documentation to be updated ## Detailed Description of the Pull Request / Additional comments Technically, the action is being done by changing the settings of the current `TerminalSettings` of the `TermControl`. Frankly, it should be operating on a copy of the `TermControl`'s `IControlSettings`, then updating the control's settings, or the Control should just listen for changes to it's setting's properties, and update in real time (without a manual call to `UpdateSettings`. However, both those paths are somewhere unknowable beyond #6904, so we'll just do this for now. ## Validation Steps Performed * tested manually with a scheme that exists * tested manually with a scheme that doesn't exist
claunia added the pull-request label 2026-01-31 09:18:24 +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#26833