microsoft / terminal

The new Windows Terminal and the original Windows console host, all in the same place!
MIT License
95.26k stars 8.27k forks source link

The default 'active' tab contrast is very low #771

Closed mikemaccana closed 4 years ago

mikemaccana commented 5 years ago

For those of us who are a little older, it's difficult to distinguish between the active and inactive tabs:

image

miniksa commented 5 years ago

@cinnamon-msft, you had a pile of designs and colors already. Do you want to put those in here so theoretically someone could work on implementing them with the XAML resources against the MUX tab control?

mdtauk commented 5 years ago

This becomes a non-issue if the Active tab reflects the same colour and styling as the console area itself. image image

zadjii-msft commented 5 years ago

@mdtauk I believe the intention is to provide that tab-styling where the tab matches the control's background as an optional (on by default?) setting.

Plus, what happens when there is more than one pane per tab? There's no way to ensure that the control is actually connected to the tab itself, so colorizing the tab with the focused control's background might look weird.

image (see #532, #702)


As far as the contrast of the tab view by default, I definitely agree that there isn't enough contrast for the light mode tabs by default. This is something we should probably upstream with the Windows UI folks building the tab control. Summoning @stmoy

mdtauk commented 5 years ago

I have commented on the design for the Tab Control when it first emerged in Sets A/B testing, and the development of it in the Community Toolkit.

Acrylic Examples x2

image

And I have always wanted to ensure Tabs felt connected, by matching them and using 60% and 90% Acrylic brushes

stmoy commented 5 years ago

@cinnamon-msft, you had a pile of designs and colors already. Do you want to put those in here so theoretically someone could work on implementing them with the XAML resources against the MUX tab control?

This is a known issue with the preview version of the Tab control that is being built in the Windows UI library. The Terminal app could temporarily override some resources to increase contrast, but this will ultimately be fixed in the Tab control implementation in the library.

eyalroth commented 5 years ago

That is one of the first things I noticed when first using the terminal. I would like to add a few screenshots of tools which I believe have a pretty good tab visual design:

VS Code tabs1

Notepad++ tabs2

Chrome tabs3

IntelliJ tabs4

stmoy commented 5 years ago

Good news, everyone!

We have heard your feedback, and the WinUI team has updated the TabView control to have significantly more contrast between the selected and rest tabs.

Here is what it will look like when the TabView ships. Note that the first tab is selected and the second one is hovered. Light Light TabView

Dark Dark TabView

DJackman123 commented 5 years ago

I think the dark version could still use more contrast.

eyalroth commented 5 years ago

@DJackman123 It actually looks much better when the image is fully enlarged.

I'm wondering though, shouldn't the focused tab be brighter than the rest? I thought the general idea in dark themes is that the brighter a surface is the more elevated it seems. Also, this is how it looks in chrome (when Windows apps are set to dark mode):

darktabs

stmoy commented 5 years ago

This is what it looks like without the hovered tab in the middle, which is more representative of the "rest state" of the control.

Dark TabView

I'm wondering though, shouldn't the focused tab be brighter than the rest?

I'm glad you asked! The general philosophy for TabView is that the selected tab and the page below it share the same color and therefore look like one contiguous surface. Since the default page color for WinUI is solid black or solid white, that means that the selected tab color must also be solid black or solid white - otherwise there would be a seam between the selected tab and the page. We're internally discussing our color ramp all-up, but for now, we're using the colors provided by the WinUI color ramp.

TabView showing no seam between tab and page

eyalroth commented 5 years ago

@stmoy In my opinion it gives a bit of a "flat" feeling to the UI.

Also the rest of Windows' UI seems to adhere to the "brighter is higher" concept: windows-dark

oceanMIH commented 5 years ago

Good news, everyone!

We have heard your feedback, and the WinUI team has updated the TabView control to have significantly more contrast between the selected and rest tabs.

Here is what it will look like when the TabView ships. Note that the first tab is selected and the second one is hovered. Light Light TabView

Dark Dark TabView

So, how soon this is available?

stmoy commented 5 years ago

So, how soon this is available?

TabView will be available in WinUI 2.2 (which we expect to ship very soon - within the next week or so).

After that, the Terminal will need to consume the release nuget package. @cinnamon-msft might be able to share more details about this part.

ghost commented 4 years ago

:tada:This issue was addressed in #3027, which has now been successfully released as Windows Terminal Preview v0.6.2951.0.:tada:

Handy links:

cf-avdd commented 4 years ago

I disagree that this is solved, at least in dark mode.

cinnamon-msft commented 4 years ago

We've opened an issue on the Microsoft UI XAML repo for accessibility purposes: https://github.com/microsoft/microsoft-ui-xaml/issues/1901

Dannymx commented 4 years ago

In dark mode my tab looks totally different than my terminal background, is there a way to set a profile property like tabColor or something like that?

image

zadjii-msft commented 4 years ago

@Dannymx you'd probably be interested in #3327

AndreyTomilinXmpi commented 1 year ago

Not sure how the issue is resolved. Here my terminal in dark mode:

image

The only difference is only bolder text, which is hard to see.

mwittmann commented 1 year ago

I agree with AndreyTomilinXmpi above. In Dark Mode, tab contrast is still too low. It is a bit better if you turn on "acrylic material" in the Settings under Appearance. image

AndreyTomilinXmpi commented 1 year ago

@mwittmann Thanks for suggestion. The acrylic didn't help much, but changing Color Scheme from "One Half Dark" to "Tango Dark" did the trick:

image

fusenuk commented 1 year ago

I'm just adding to the above comments, this doesn't feel fixed when using the 'Dark' overall theme and certain Colour Schemes that have fairly dark backgrounds.

I've worked around it by changing the overall theme to 'Light' and leaving my Colour Scheme as a dark colour, but there should be a setting for the difference in contrast.

jimyag commented 1 month ago

https://github.com/microsoft/terminal/discussions/15675 Setting the active tab color worked for me image

"theme": "jimyag",
    "themes": [
        {
            "name": "jimyag",
            "window": {
                "applicationTheme": "dark"
            },
            "tab": {
                "background": "#A2734C",
                "unfocusedBackground": "#000"
            }
        }
    ],