[1.17] Setting app theme results in mismatched light/dark colors for SUI text #19074

Closed
opened 2026-01-31 06:33:03 +00:00 by claunia · 3 comments
Owner

Originally created by @carlos-zamora on GitHub (Dec 15, 2022).

Originally assigned to: @zadjii-msft on GitHub.

Go to Appearance -> set theme to 'Light'

Some settings have the header in black some have them in white (seems to be the different setting container styles)
GetImage (3)

Originally created by @carlos-zamora on GitHub (Dec 15, 2022). Originally assigned to: @zadjii-msft on GitHub. Go to Appearance -> set theme to 'Light' Some settings have the header in black some have them in white (seems to be the different setting container styles) <img width="761" alt="GetImage (3)" src="https://user-images.githubusercontent.com/11050425/207741120-6d178b39-6d12-4161-838e-e72bacca916b.png">
Author
Owner

@zadjii-msft commented on GitHub (Jan 6, 2023):

Hmm. I don't know if I can repro this exactly.

What I can repro is the SUI BG not matching the app theme. That I think regressed in #14567, which I can't remember if it made it to the bug bash or not.

Does this happen on launch when the theme is set to "light"? Or do you need to switch from some other theme?

I'll submit the mica fix, and then you try that out and tell me if this is gone

@zadjii-msft commented on GitHub (Jan 6, 2023): Hmm. I don't know if I can repro this exactly. What I _can_ repro is the SUI BG not matching the app theme. That I think regressed in #14567, which I can't remember if it made it to the bug bash or not. Does this happen on launch when the theme is set to "light"? Or do you need to switch from some other theme? I'll submit the mica fix, and then you try that out and tell me if this is gone
Author
Owner

@carlos-zamora commented on GitHub (Jan 10, 2023):

Hmm. I don't know if I can repro this exactly.

What I can repro is the SUI BG not matching the app theme. That I think regressed in #14567, which I can't remember if it made it to the bug bash or not.

Does this happen on launch when the theme is set to "light"? Or do you need to switch from some other theme?

I'll submit the mica fix, and then you try that out and tell me if this is gone

The way I got it to repro was...

  1. OS theme = dark
  2. In Terminal, start with theme=terminal
  3. switch to theme=Light
        {
            "name": "terminal",
            "tab": 
            {
                "background": "terminalBackground",
                "showCloseButton": "always",
                "unfocusedBackground": null
            },
            "tabRow": 
            {
                "background": "terminalBackground",
                "unfocusedBackground": null
            },
            "window": 
            {
                "applicationTheme": "dark",
                "useMica": false
            }
        }
@carlos-zamora commented on GitHub (Jan 10, 2023): > Hmm. I don't know if I can repro this exactly. > > What I _can_ repro is the SUI BG not matching the app theme. That I think regressed in #14567, which I can't remember if it made it to the bug bash or not. > > Does this happen on launch when the theme is set to "light"? Or do you need to switch from some other theme? > > I'll submit the mica fix, and then you try that out and tell me if this is gone The way I got it to repro was... 1. OS theme = dark 1. In Terminal, start with theme=terminal 2. switch to theme=Light ```js { "name": "terminal", "tab": { "background": "terminalBackground", "showCloseButton": "always", "unfocusedBackground": null }, "tabRow": { "background": "terminalBackground", "unfocusedBackground": null }, "window": { "applicationTheme": "dark", "useMica": false } } ```
Author
Owner

@ghost commented on GitHub (Jan 24, 2023):

:tada:This issue was addressed in #14644, which has now been successfully released as Windows Terminal Preview v1.17.1023.🎉

Handy links:

@ghost commented on GitHub (Jan 24, 2023): :tada:This issue was addressed in #14644, which has now been successfully released as `Windows Terminal Preview v1.17.1023`.:tada: Handy links: * [Release Notes](https://github.com/microsoft/terminal/releases/tag/v1.17.1023) * [Store Download](https://www.microsoft.com/store/apps/9n8g5rfz9xk3?cid=storebadge&ocid=badge)
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#19074