-
Notifications
You must be signed in to change notification settings - Fork 8.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add a --colorScheme
param to new-tab
, split-pane
#9602
Conversation
This comment has been minimized.
This comment has been minimized.
@@ -66,6 +66,10 @@ namespace winrt::Microsoft::Terminal::Settings::Model::implementation | |||
const til::color tabColor{ _TabColor.Value() }; | |||
ss << fmt::format(L"tabColor: {}, ", tabColor.ToHexString(true)); | |||
} | |||
if (!_ColorScheme.empty()) | |||
{ | |||
ss << fmt::format(L"colorScheme: {}, ", _ColorScheme); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Might it make sense to use FMT_COMPILE here (and the one below) instead similar to #9581 or would it make more sense to do all the params at once in a separate PR?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Probably makes sense to do them all in a single pass IMO
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
My notes should have come with an approval.
Hello @zadjii-msft! Because this pull request has the p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (
|
🎉 Handy links: |
This is entirely self-serving. In my go-to config, I like having some of
the panes for a given profile in a different color scheme. This will let
a user pass
--colorScheme <scheme name>
to manually override thescheme for that profile. Neat!