Skip to content

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

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

Tab background on active/hovered is dark even in light mode #14815

Closed
Octal450 opened this issue Feb 9, 2023 · 13 comments
Closed

Tab background on active/hovered is dark even in light mode #14815

Octal450 opened this issue Feb 9, 2023 · 13 comments
Labels
Area-Theming Anything related to the theming of elements of the window Needs-Triage It's a new issue that the core contributor team needs to triage at the next triage meeting Product-Terminal The new Windows Terminal. Resolution-By-Design It's supposed to be this way. Sometimes for compatibility reasons.

Comments

@Octal450
Copy link

Octal450 commented Feb 9, 2023

Windows Terminal version

1.16.10262.0

Windows build number

10.0.22621.1194

Other Software

No response

Steps to reproduce

  • Open Windows terminal when the light them OR use system theme (with Windows set to light mode)

  • Observe the tab row
    image

  • Hover over a tab while another tab is open:
    image

Expected Behavior

Like in the previous version, the tabs should be light when in light mode (like Settings is)
image

Actual Behavior

The tabs (except for settings) are dark when in light mode.

@Octal450 Octal450 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 Feb 9, 2023
@Octal450
Copy link
Author

Octal450 commented Feb 9, 2023

PS: I noticed that by default the application was now in "Dark" mode as opposed to "Windows default". Why? Shouldn't it track Windows' color scheme by default?

@CirnoT
Copy link

CirnoT commented Feb 10, 2023

I believe this might be desired behavior. Tab color now tracks background color of selected color scheme for given profile.

WindowsTerminal_2023-02-10_01-31-36
WindowsTerminal_2023-02-10_01-32-13
WindowsTerminal_2023-02-10_01-32-27

Can't say I particularly agree with this choice but I also very much doubt it's gonna be reverted or changed.

@j4james
Copy link
Collaborator

j4james commented Feb 10, 2023

This behavior is configurable with the new theme settings. See the documentation here:
https://learn.microsoft.com/en-us/windows/terminal/customize-settings/themes

@Octal450
Copy link
Author

That's rather silly... I don't understand the reasoning for such functionality at all. Especially being on by default...

@3tothe6
Copy link

3tothe6 commented Feb 10, 2023

This may be a bug introduced in v1.16 in which theme support was stabilized. For people who don't have time reading the doc, backup your settings.json and revert to v1.15.3465.0, then your problem may be solved.

@CirnoT
Copy link

CirnoT commented Feb 10, 2023

This behavior is configurable with the new theme settings. See the documentation here: https://learn.microsoft.com/en-us/windows/terminal/customize-settings/themes

It is actually impossible to redefine built-in themes as they are removed from JSON file; so "Use system theme" behavior can not be reproduced.

For those looking to reproduce old behavior; see attached theme configuration:

{
    "themes": 
    [
        {
            "name": "Dark (legacy)",
            "tab": 
            {
                "background": null,
                "unfocusedBackground": null
            },
            "window": 
            {
                "applicationTheme": "dark"
            }
        },
        {
            "name": "Light (legacy)",
            "tab": 
            {
                "background": null,
                "unfocusedBackground": null
            },
            "window": 
            {
                "applicationTheme": "light"
            }
        },
        {
            "name": "Use Windows theme (legacy)",
            "tab": 
            {
                "background": null,
                "unfocusedBackground": null
            }
        }
    ]
}

@zadjii-msft
Copy link
Member

zadjii-msft commented Feb 10, 2023

As noted in this thread, we introduced support for Themes in 1.16 (and further refined them in 1.17). As a part of that work, we took the opportunity to revisit what the "out-of-the-box" theme of the Terminal looks like1. We really wanted to revisit the fact that for most users (who haven't set the OS theme), when they launch the Terminal, they get a window that's mostly black, with a white titlebar. This contrast seemed a little silly to us. So we opted to optimize for the first run experience being dark theme, a dark-gray titlebar with tabs that were seamless (effectively, black).

Of course, customizability has been one of my zealous ideals here on the Terminal, so while changing the default themes, I wanted to make sure to give folks whatever knobs they needed to make the Terminal look however they like.

The situation we probably failed to consider as deeply as we should was folks who had a light titlebar, and wanted to just use the new "light" theme out of the box. As noted in this thread - the default light theme with the default dark colorscheme creates a black terminal, with a black tab, on a white titlebar. Clearly, there's a lot of folks here who don't love this look2.

I've maybe overestimated in my head how willing folks would be to change the theme settings if they didn't like the defaults. That's on me - I'm always tinkerering with my settings, but my experience is not universal. Most folks just want sane defaults. I had made a lot of assumptions and let my personal preferences get involved here, so that's my bad.

I think we should more broadly promote the "legacy" themes as authored by @CirnoT in this thread. Those are just about what the old themes looked like, so if that's what people want, I wanna give the people what they want. I'd like to leave this thread open to track ingesting those into the default list of themes. Now, that might be a bit tricky. The built-in theme names need to get localized, and I don't know if we can check in loc changes like this in a servicing release. If we can't, then they'd be first (re)-available in 1.17 Stable/1.18 Preview.

We're discussing some options as a team, but probably won't have more to share till after we discuss more on Monday3.

In the meantime, I'd recommend everyone who's missing the old themes insert that blob into their settings.json.

Thanks for all the lively feedback here!

Footnotes

  1. This had already changed slightly over the last couple releases, as we were working on ingesting some of the WinUI changes for Windows 11.

  2. Though, I do think this is where we're getting into a discussion of UX taste - in discussion we had as a team, we generally preferred the new light theme with the seamless black tab, over the old "black terminal, white tab, light titlebar" look of the old theme. Though the white tab/light titlebar does look a lot like the vintage console...

  3. EDIT: some conflicts came up and our team sync is getting moved this week, though I'm not quite sure when it'll be yet.

@zadjii-msft zadjii-msft added Resolution-By-Design It's supposed to be this way. Sometimes for compatibility reasons. Product-Terminal The new Windows Terminal. Needs-Discussion Something that requires a team discussion before we can proceed Area-Theming Anything related to the theming of elements of the window and removed Issue-Bug It either shouldn't be doing this or needs an investigation. labels Feb 10, 2023
@Octal450 Octal450 reopened this Feb 10, 2023
@Octal450
Copy link
Author

Octal450 commented Feb 10, 2023

@zadjii-msft ,
Thank you for the detailed reply and time taken. I appreciate it greatly.

I do understand your reasoning, especially given that the dark mode has taken off in popularity (especially among techies like us) quite greatly over the last few years.

However, the issue is with a system in light mode, having one window with a dark titlebar looks horribly out of place, while all the other windows regardless of color have a white titlebar - which fits the white UI of most applications on a system with light mode selected (which I would like to point out, is the default that Windows ships with).

Not to mention the consistency nightmare of a light titlebar with dark tabs when "light" mode is selected.

Personally, I don't see the issue of a light titlebar with a dark command window, given that it has been this way for roughly 25 years. I strongly believe that respecting the operating system's defaults should be the highest priority in order to provide a seamless and consistent user experience.

Kind Regards,
Josh

PS: Apologies for the close - misclicked.

@zadjii-msft
Copy link
Member

I strongly believe that respecting the operating system's defaults should be the highest priority in order to provide a seamless and consistent user experience.

You seem like the kind of person who might like what we've got in store in 1.17: (warning, this gif was taken on a debug build - it's faster on a release build)

@CirnoT
Copy link

CirnoT commented Feb 10, 2023

I've maybe overestimated in my head how willing folks would be to change the theme settings if they didn't like the defaults. That's on me - I'm always tinkerering with my settings, but my experience is not universal. Most folks just want sane defaults. I had made a lot of assumptions and let my personal preferences get involved here, so that's my bad.

Speaking from experience - I was very much surprised to see Terminal suddenly changed to a dark theme and sought out to see what caused it. While investigating and switching to light theme, the initial impression I had when tab color remained dark is that there is something wrong.

Note that updates for Terminal are now completely seamless and happen in background. I have not even rebooted my PC and was very much unaware of the update happening until I checked it. Maybe this is part of the issue here, especially with switching existing users to dark with no update prompt or heads-up (a simple "What's new" message on first run might've been sufficient to clear some misunderstandings)

@Octal450
Copy link
Author

Hi,
That does look nifty.

Personally, I am not so worried about the color of the terminal body itself, especially being used to a dark terminal from all my years of using various operating systems (except macOS... which has a light one)

For me personally, it was more regarding the titlebar/tabs/window decorations. Working in the light mode environment develops "reflex" like responses when wanting to resize, close, minimize, etc as a result of the window decorations being consistent regardless of the application (well... except special programs which provide their own, but you get the idea).

As a result, I find myself fumbling now and then using the terminal in dark mode, being the sole window who's titlebar has a different appearance without using a custom title bar.

An interesting sidenote, it also seems the new Photos app in Windows 11 has the exact same - a default of dark mode rather than respecting the color settings.

Kind Regards,
Josh

@fxzxmic
Copy link

fxzxmic commented Feb 11, 2023

@zadjii-msft I know that you want to provide a seamless experience of tabs and interfaces, and you have done it. This is a great idea.
However, I hope to consider the diversity of user needs. And the dark tabs and light tabs' bar actually aggravate the sense of fragmentation. Fortunately, it does not cause much trouble because it retains the old experience.

Thank @CirnoT for the method to restore the old experience.

@Limyx826
Copy link

Suprise me when the first thing that happen when I open Terminal is seeing dark theme. Even changing into light theme also have some of the dark theme elements left over.

@microsoft microsoft locked and limited conversation to collaborators Feb 13, 2023
@carlos-zamora carlos-zamora converted this issue into discussion #14844 Feb 13, 2023
@carlos-zamora carlos-zamora removed the Needs-Discussion Something that requires a team discussion before we can proceed label Feb 21, 2023

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

Labels
Area-Theming Anything related to the theming of elements of the window Needs-Triage It's a new issue that the core contributor team needs to triage at the next triage meeting Product-Terminal The new Windows Terminal. Resolution-By-Design It's supposed to be this way. Sometimes for compatibility reasons.
Projects
None yet
Development

No branches or pull requests

8 participants