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

Update tab hover colors #13434

Merged
merged 2 commits into from Jul 5, 2022
Merged

Update tab hover colors #13434

merged 2 commits into from Jul 5, 2022

Conversation

ghost
Copy link

@ghost ghost commented Jul 5, 2022

The hover tab color used to be generated from the selected tab color, which would end up lighter or darker, and white-gray colors would end up pink.
It is now simply the selected tab color with 60% opacity. This is also how brushes are created for accent buttons and color buttons (although with different opacity levels).

Copy link
Member

@zadjii-msft zadjii-msft left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:chef_kiss:

Copy link
Member

@DHowett DHowett left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I love it. Thanks!

@DHowett
Copy link
Member

DHowett commented Jul 5, 2022

@zadjii-msft /propose 1.15

@zadjii-msft
Copy link
Member

meh, I haven't checked this out locally, just the code eyeball test. Lemme give it a once over before I give it the 1.15 👍

@ghost
Copy link
Author

ghost commented Jul 5, 2022

A last minute change. I just realised that by using opacity instead of alpha you would be able to use a semi-transparent color for the selected tab (before it would override the alpha).
Semi-transparent colors are still not supported, but that can be added in the future.

@zadjii-msft
Copy link
Member

:shipit:

@DHowett
Copy link
Member

DHowett commented Jul 5, 2022

Ah, very cool. How does it look now with the slightly modified opacity values, as well?

@DHowett
Copy link
Member

DHowett commented Jul 5, 2022

Images removed from body of PR

Normal/hover/selected, before/after:
image
image

@DHowett
Copy link
Member

DHowett commented Jul 5, 2022

I'm actually okay with these options. Custom tab colors are already pretty niche, and folks probably won't be keeping a bunch of tabs of the same color next to each other long enough to be annoyed by the similarity.

@zadjii-msft, what do you think about the new ramp?

@zadjii-msft
Copy link
Member

I only tested the new ramp, and it seemed good to me TBH.

@DHowett
Copy link
Member

DHowett commented Jul 5, 2022

I'll merge this now for 1.15's cut. @dansmor7, if you want to play with it some more I'm happy to bring that into 1.16 or a later servicing release of 1.15. Thanks for the really quick feedback loop! :)

@DHowett DHowett added the AutoMerge Marked for automatic merge by the bot when requirements are met label Jul 5, 2022
@ghost
Copy link

ghost commented Jul 5, 2022

Hello @DHowett!

Because this pull request has the AutoMerge label, I will be glad to assist with helping to merge this pull request once all check-in policies pass.

Do note that I've been instructed to only help merge pull requests of this repository that have been opened for at least 8 hours, a condition that will be fulfilled in about 5 hours 40 minutes. No worries though, I will be back when the time is right! 😉

p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (@msftbot) and give me an instruction to get started! Learn more here.

@DHowett DHowett merged commit c6b67aa into microsoft:main Jul 5, 2022
DHowett pushed a commit that referenced this pull request Jul 5, 2022
The hover tab color used to be generated from the selected tab color, which would end up lighter or darker, and white-gray colors would end up pink.
It is now simply the selected tab color with 60% opacity. This is also how brushes are created for accent buttons and color buttons (although with different opacity levels).

(cherry picked from commit c6b67aa)
Service-Card-Id: 83894208
Service-Version: 1.15
@ghost
Copy link

ghost commented Jul 6, 2022

🎉Windows Terminal Preview v1.15.186 has been released which incorporates this pull request.:tada:

Handy links:

@ghost ghost deleted the hover-tab-color branch September 1, 2022 20:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AutoMerge Marked for automatic merge by the bot when requirements are met
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants