Skip to content

Change SettingsTab close icon to be smaller - #9324

Merged
1 commit merged into
mainfrom
dev/cazamor/sui/settings-tab-close-icon
Mar 4, 2021
Merged

1 commit merged into
mainfrom
dev/cazamor/sui/settings-tab-close-icon

Conversation

@carlos-zamora

Copy link
Copy Markdown

We were using the wrong close icon for the settings tab. This is a side effect of TerminalTab overriding TabBase's implementation.

#6800 - Settings UI Epic

Closes #9317

@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 Mar 2, 2021
@DHowett Dustin L. Howett (DHowett) added the AutoMerge Marked for automatic merge by the bot when requirements are met label Mar 4, 2021
@ghost

Copy link
Copy Markdown

Hello Dustin L. Howett (@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.

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 4a95341 into main Mar 4, 2021
@ghost
ghost deleted the dev/cazamor/sui/settings-tab-close-icon branch March 4, 2021 19:32
Dustin L. Howett (DHowett) pushed a commit that referenced this pull request Mar 15, 2021
We were using the wrong close icon for the settings tab. This is a side effect of `TerminalTab` overriding `TabBase`'s implementation.

#6800 - Settings UI Epic

Closes #9317

(cherry picked from commit 4a95341)
Dustin L. Howett (DHowett) pushed a commit that referenced this pull request Mar 15, 2021
We were using the wrong close icon for the settings tab. This is a side effect of `TerminalTab` overriding `TabBase`'s implementation.

#6800 - Settings UI Epic

Closes #9317

(cherry picked from commit 4a95341)
@ghost

Copy link
Copy Markdown

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

Handy links:

@ghost

Copy link
Copy Markdown

🎉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.

The Settings context menu still uses the BIG close icon

3 participants