Skip to content

Give our NavigationView's acrylic a fallback color#9752

Merged
1 commit merged into
mainfrom
dev/duhowett/we're_routed_fall_back
Apr 9, 2021
Merged

Give our NavigationView's acrylic a fallback color#9752
1 commit merged into
mainfrom
dev/duhowett/we're_routed_fall_back

Conversation

@DHowett
Copy link
Copy Markdown
Member

@DHowett DHowett commented Apr 8, 2021

It will be a different color than the background, so it will look less
weird when it's unfocused. It also fixes the bug where the navigation
menu is transparent when acrylic is disabled systemwide.

Fixes #9337

It will be a different color than the background, so it will look less
weird when it's unfocused. It also fixes the bug where the navigation
menu is transparent when acrylic is disabled systemwide.

Fixes #9337
@ghost ghost added Area-SettingsUI Anything specific to the SUI Issue-Bug It either shouldn't be doing this or needs an investigation. Priority-3 A description (P3) Product-Terminal The new Windows Terminal. labels Apr 8, 2021
@DHowett
Copy link
Copy Markdown
Member Author

DHowett commented Apr 8, 2021

This looks like ...

image

instead of ...

image

I did not want it to match the titlebar. It is not part of the title bar. It becoming "lighter" matches what happens when acrylic is on (though in Light theme, it becomes grayer ... which is also what happens when acrylic is on)

@DHowett DHowett added the Needs-Second It's a PR that needs another sign-off label Apr 8, 2021
@zadjii-msft zadjii-msft added the AutoMerge Marked for automatic merge by the bot when requirements are met label Apr 9, 2021
@ghost
Copy link
Copy Markdown

ghost commented Apr 9, 2021

Hello @zadjii-msft!

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.

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.

@ghost ghost merged commit b310b1c into main Apr 9, 2021
@ghost ghost deleted the dev/duhowett/we're_routed_fall_back branch April 9, 2021 10:43
DHowett added a commit that referenced this pull request Apr 12, 2021
It will be a different color than the background, so it will look less
weird when it's unfocused. It also fixes the bug where the navigation
menu is transparent when acrylic is disabled systemwide.

Fixes #9337

(cherry picked from commit b310b1c)
@ghost
Copy link
Copy Markdown

ghost commented Apr 14, 2021

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

Handy links:

@ghost
Copy link
Copy Markdown

ghost commented Apr 14, 2021

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

Handy links:

This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area-SettingsUI Anything specific to the SUI AutoMerge Marked for automatic merge by the bot when requirements are met Issue-Bug It either shouldn't be doing this or needs an investigation. Needs-Second It's a PR that needs another sign-off Priority-3 A description (P3) Product-Terminal The new Windows Terminal.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Settings UI navigation view has a transparent background when opened as an overlay

3 participants