Skip to content
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

[1.17] Dark titlebar in light theme #14732

Closed
angelog0 opened this issue Jan 25, 2023 · 12 comments
Closed

[1.17] Dark titlebar in light theme #14732

angelog0 opened this issue Jan 25, 2023 · 12 comments
Labels
Area-Theming Anything related to the theming of elements of the window Issue-Bug It either shouldn't be doing this or needs an investigation. Product-Terminal The new Windows Terminal. Resolution-By-Design It's supposed to be this way. Sometimes for compatibility reasons. Resolution-Fix-Available It's available in an Insiders build or a release

Comments

@angelog0
Copy link

Windows Terminal version

1.17.1023

Windows build number

10.0.19045.2546

Other Software

No response

Steps to reproduce

Start WT. it shows up as in the attachment, i.e. with a black background in the window title bar

wt_start

Expected Behavior

After I click on another window app or on the Desktop it shows up as in the attachment which I would expect and prefer , i.e. with a white background in the window title bar. Notice WT does not return to look as in the first start above (black background in window title bar)

wt_after

Actual Behavior

See above.

@angelog0 angelog0 added Issue-Bug It either shouldn't be doing this or needs an investigation. Needs-Triage It's a new issue that the core contributor team needs to triage at the next triage meeting labels Jan 25, 2023
@DHowett
Copy link
Member

DHowett commented Jan 25, 2023

Would you mind sharing your settings.json?

@DHowett DHowett added the Needs-Author-Feedback The original author of the issue/PR needs to come back and respond to something label Jan 25, 2023
@angelog0
Copy link
Author

Now attached.

wt-settings.tar.gz

@ghost ghost added Needs-Attention The core contributors need to come back around and look at this ASAP. and removed Needs-Author-Feedback The original author of the issue/PR needs to come back and respond to something labels Jan 25, 2023
@angelog0
Copy link
Author

BTW, I had (just now uninstalled) WT-1.15 (no-preview) and it had changed my settings.json removing ALL my comments and adding some other stuff SILENTLY. Luckily I have a backup of that file... I often leave as comment old configurations and alternative to statements and references to WEB and other docs where I find suggestions etc..

@carlos-zamora carlos-zamora removed the Needs-Attention The core contributors need to come back around and look at this ASAP. label Jan 25, 2023
@carlos-zamora carlos-zamora added this to the Terminal v1.18 milestone Jan 25, 2023
@carlos-zamora carlos-zamora added Product-Terminal The new Windows Terminal. Area-Theming Anything related to the theming of elements of the window and removed Needs-Triage It's a new issue that the core contributor team needs to triage at the next triage meeting labels Jan 25, 2023
@zadjii-msft
Copy link
Member

@angelog0 Thet's tracked in #8991

@zadjii-msft
Copy link
Member

        {
            "name": "system2",
            "window": {
                "applicationTheme": "system"
            },
            "tab": {
                "background": "terminalBackground",
                "showCloseButton": "hover",
                "unfocusedBackground": "#00000000"
            }
        },

I'd reckon this is due to your OS being in dark theme, the Terminal being in system, and us finally obeying the app theme when it comes to the titlebar (#14536).

If you change system to light in your theme, is that more what you're looking for/?

@ghost ghost added the Needs-Author-Feedback The original author of the issue/PR needs to come back and respond to something label Jan 25, 2023
@zadjii-msft zadjii-msft added the Needs-Triage It's a new issue that the core contributor team needs to triage at the next triage meeting label Jan 25, 2023
@angelog0
Copy link
Author

@zadjii-msft I tried this:

{
            "name": "system2",
            "window": {
                "applicationTheme": "light"
            },
            "tab": {
                "background": "terminalBackground",
                "showCloseButton": "hover",
                "unfocusedBackground": "#00000000"
            }
        },

but it does not work: same result...

@ghost ghost added Needs-Attention The core contributors need to come back around and look at this ASAP. and removed Needs-Author-Feedback The original author of the issue/PR needs to come back and respond to something labels Jan 26, 2023
@zadjii-msft zadjii-msft changed the title Strange behavior after 1.17 preview release [1.17] Dark titlebar in light theme Jan 29, 2023
@carlos-zamora carlos-zamora removed Needs-Triage It's a new issue that the core contributor team needs to triage at the next triage meeting Needs-Attention The core contributors need to come back around and look at this ASAP. labels Feb 1, 2023
@zadjii-msft
Copy link
Member

10.0.19045.2546

Hmmmmmmm the API we use to change the titlebar color might not exist in Windows 10. Any chance you've got a Windows 11 machine you could test with?

@zadjii-msft
Copy link
Member

DWMWA_USE_IMMERSIVE_DARK_MODE
Use with DwmSetWindowAttribute. Allows the window frame for this window to be drawn in dark mode colors when the dark mode system setting is enabled. For compatibility reasons, all windows default to light mode regardless of the system setting. The pvAttribute parameter points to a value of type BOOL. TRUE to honor dark mode for the window, FALSE to always use light mode.

This value is supported starting with Windows 11 Build 22000.

emph. added

I think I'm gonna have to call this one by-design. I can't repro this with your theme on Win11, and it looks like that API is only officially supported on Win 11.

@zadjii-msft zadjii-msft added the Resolution-By-Design It's supposed to be this way. Sometimes for compatibility reasons. label Apr 4, 2023
@microsoft-github-policy-service microsoft-github-policy-service bot added the Needs-Tag-Fix Doesn't match tag requirements label Apr 4, 2023
@zadjii-msft zadjii-msft added Resolution-Fix-Available It's available in an Insiders build or a release and removed Needs-Tag-Fix Doesn't match tag requirements labels Apr 4, 2023
@angelog0

This comment was marked as off-topic.

@angelog0

This comment was marked as off-topic.

@DHowett
Copy link
Member

DHowett commented Apr 5, 2023

Your comment about the system requirements of Windows 11 and the size of our heads IS off-topic, @angelog0. The topic of this issue is the titlebar color, and the topic of this repository is the Windows Terminal and Windows Console Host. If you want a forum in which to air your grievances about things that are not Windows Terminal, the Windows Console Host, or the titlebar color of Windows Terminal or the Windows Console Host I will gladly redirect you to Reddit.

@microsoft microsoft locked and limited conversation to collaborators Apr 5, 2023
@DHowett
Copy link
Member

DHowett commented Apr 5, 2023

(This change you're asking about actually isn't version-specific. Your tab row is set to light theme, which suggests that your application is set to light theme. I think we should offer title bar customization alongside tab row customization probably. We found that out while triaging a later issue than this one.)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Area-Theming Anything related to the theming of elements of the window Issue-Bug It either shouldn't be doing this or needs an investigation. Product-Terminal The new Windows Terminal. Resolution-By-Design It's supposed to be this way. Sometimes for compatibility reasons. Resolution-Fix-Available It's available in an Insiders build or a release
Projects
None yet
Development

No branches or pull requests

4 participants