Skip to content

Fix setting launch position in the Settings UI not working#14518

Merged
1 commit merged intomainfrom
dev/pabhoj/fix_launch_params
Dec 9, 2022
Merged

Fix setting launch position in the Settings UI not working#14518
1 commit merged intomainfrom
dev/pabhoj/fix_launch_params

Conversation

@PankajBhojwani
Copy link
Contributor

Summary of the Pull Request

This change should have been a part of #14190 but was missed.

PR Checklist

Validation Steps Performed

Setting launch position in the settings UI works now

@ghost ghost added Area-SettingsUI Anything specific to the SUI Issue-Bug It either shouldn't be doing this or needs an investigation. Priority-1 A description (P1) Product-Terminal The new Windows Terminal. Severity-Blocking We won't ship a release like this! No-siree. labels Dec 8, 2022
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.

WAIT REALLY? THATS IT‽

@carlos-zamora carlos-zamora added the AutoMerge Marked for automatic merge by the bot when requirements are met label Dec 9, 2022
@ghost
Copy link

ghost commented Dec 9, 2022

Hello @carlos-zamora!

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 b9c3b5c into main Dec 9, 2022
@ghost ghost deleted the dev/pabhoj/fix_launch_params branch December 9, 2022 17:51
@DHowett
Copy link
Member

DHowett commented Dec 9, 2022

@PankajBhojwani was launch position UI part of 1.16?

@ianjoneill
Copy link
Contributor

There's still something a bit funky with this:

LaunchPosition.mp4

@ianjoneill
Copy link
Contributor

^ This is fixed in #14522

ghost pushed a commit that referenced this pull request Dec 13, 2022
)

Ensures the X & Y XAML controls are updated when the default launch position checkbox is toggled.
References #14518
@ghost
Copy link

ghost commented Jan 24, 2023

🎉Windows Terminal Preview v1.17.1023 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. Priority-1 A description (P1) Product-Terminal The new Windows Terminal. Severity-Blocking We won't ship a release like this! No-siree. zBugBash-Consider

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1.17 still displays launch params (Default,Default) despite values being set

5 participants