-
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
New terminal does not handle theme change event. #1230
Comments
This one is interesting. We could probably tackle it as part of #1125, but maybe we want to keep it separate? |
The window border as well as the area with the Minimize, Maximize and Close buttons does not take the color of the theme (remains white despite the dark theme). |
same here , anyone know how to fix this? |
To be blunt: somebody needs to write the code to do this. If we do it, it will come out whenever we are done. If we don’t, we will get back to it eventually. |
@DHowett May I work on it?? |
@AnuthaDev you certainly may 😄 |
Yes please change the window title according to the dark/light theme in Windows. I also have to close and re-open the terminal to fully switch. The theme itself I change the settings.json at the moment, configuring two themes, one for dark, another for light and let it auto-switch according to Windows theme would be even better. |
Not sure if this is a XAML Island thing, but if you are using Xaml ThemeResources - they should update and refresh as the RequestedTheme changes. |
This is almost certainly Islands related. I'm not sure what window message is sent when the theme changes, but there almost certainly should be one... |
@zadjii-msft I think it's this one: WM_SETTINGCHANGE |
can you share how you did this? |
@Robinzon100 my theme switching is done in this bash script I would prefer to just configure a dark and light theme in the terminal and let it update automatically depending on the system theme setting in the registry. (Update: fixed the link to my script) |
I would guess the
(but using 1 if the current value is 0) and also JSON-editing settings.json for Terminal to change the theme, which WT will immediately apply when it notices the file change. Edit: Beat me to it by 30 seconds. |
Hi , is there any update on fix? How can we get theme auto switching and config for default light and default dark themes ? |
Given that the issue is labelled "Help Wanted", I think it's safe to assume that's a yes. 😉 If you'd like to help, let the team know that you want to work on the feature, and explain briefly how you intend to implement it. You can find more information in the Contributor's Guide. |
Yep, there's no real reason we haven't done this other than simply resourcing. We've only got so many devs, and something like 10,000 issues that have been active since the Terminal was first released 😄 |
Terminal.on.Win11.mp4For some reason Windows Terminal 1.8.1521.0 follows system theme on Windows 11, but not on Windows 10. |
There is a bug, so this does not work yet (as of v1.11): microsoft/terminal#1230.
Huh, that's totally right, this did seemingly get fixed in Windows 11. That seems to me to imply that there was actually just an OS bug that was the actual root cause for this one. I'm gonna close this one out as external - there's maybe something that we could do to fix this on Windows 10, but the fact that an OS fix remedied this makes me think that would be a futile effort. |
@zadjii-msft I actually have a fix for this on Windows 10 which is not that complicated. Is it worth submitting a PR, or would you rather just leave it as is? |
@j4james If you have a fix in hand then go for it! I'm maybe being a little aggressive in closing out some of the older things in the backlog. |
From MSFT:21424988
I have to restart the new terminal to see the effects of changing between light and dark windows theme.
You can see at the attached screenshot that the tabs haven't changed it's colors after I changed from Dark to Light theme. UWP apps are supposed to do that by default.
The text was updated successfully, but these errors were encountered: