Skip to content

Fix focus box around color schemes combo box#12439

Merged
DHowett merged 2 commits intomainfrom
dev/pabhoj/textbox_border
Feb 8, 2022
Merged

Fix focus box around color schemes combo box#12439
DHowett merged 2 commits intomainfrom
dev/pabhoj/textbox_border

Conversation

@PankajBhojwani
Copy link
Contributor

@PankajBhojwani PankajBhojwani commented Feb 8, 2022

Summary of the Pull Request

The focus box around the color schemes combo box was getting cut off, this change adds a small margin to the stackpanel to allow space for the focus box

PR Checklist

Validation Steps Performed

combobox

@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 Feb 8, 2022

<!-- Select Color and Add New Button -->
<StackPanel Orientation="Horizontal">
<StackPanel Margin="0,5,0,0"
Copy link
Member

Choose a reason for hiding this comment

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

this also moves the whole box down -- OK?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, very slightly! The page doesn't look any different from the other pages though (and it is going to be overhauled very soon anyway)

@DHowett DHowett merged commit f5f0712 into main Feb 8, 2022
@DHowett DHowett deleted the dev/pabhoj/textbox_border branch February 8, 2022 22:35
DHowett pushed a commit that referenced this pull request Feb 9, 2022
The focus box around the color schemes combo box was getting cut off, this change adds a small margin to the stackpanel to allow space for the focus box

## PR Checklist
* [x] Closes #12328

(cherry picked from commit f5f0712)
@ghost
Copy link

ghost commented Feb 11, 2022

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

Handy links:

zadjii-msft pushed a commit that referenced this pull request Mar 3, 2022
The focus box around the color schemes combo box was getting cut off, this change adds a small margin to the stackpanel to allow space for the focus box

## PR Checklist
* [x] Closes #12328
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 Issue-Bug It either shouldn't be doing this or needs an investigation. Priority-3 A description (P3) Product-Terminal The new Windows Terminal.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Focused Color schemes not showing top border

3 participants