Skip to content

Re-evaluate the size of the quake window when it's summoned to a monitor#10674

Merged
1 commit merged intomainfrom
dev/migrie/b/10274-10182-reevaluate-quake-size
Jul 20, 2021
Merged

Re-evaluate the size of the quake window when it's summoned to a monitor#10674
1 commit merged intomainfrom
dev/migrie/b/10274-10182-reevaluate-quake-size

Conversation

@zadjii-msft
Copy link
Member

@zadjii-msft zadjii-msft commented Jul 15, 2021

Summary of the Pull Request

When the quake window is moved to another monitor, re-evaluate it's size for that monitor.

References

PR Checklist

Detailed Description of the Pull Request / Additional comments

We'll probably need to do this in a few more places, but I'm breaking PRs into small chunks for easier reviews.

Validation Steps Performed

Summoned the window to a bunch of different resolutions. Where it would use the wrong size before, it no longer does.

  I can't believe that we never did this. We'd just leave the quake
  window's size totally unchanged. That's so ridiculously dumb.
@ghost ghost added Area-UserInterface Issues pertaining to the user interface of the Console or Terminal Issue-Bug It either shouldn't be doing this or needs an investigation. Priority-2 A description (P2) Product-Terminal The new Windows Terminal. labels Jul 15, 2021
@zadjii-msft
Copy link
Member Author

Wait no, this doesn't fix #10274 (comment). Hmm.

Copy link
Member

@miniksa miniksa left a comment

Choose a reason for hiding this comment

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

Yep OK.

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

ghost commented Jul 20, 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 9c1331a into main Jul 20, 2021
@ghost ghost deleted the dev/migrie/b/10274-10182-reevaluate-quake-size branch July 20, 2021 16:26
DHowett pushed a commit that referenced this pull request Aug 25, 2021
…tor (#10674)

## Summary of the Pull Request

When the quake window is moved to another monitor, re-evaluate it's size for that monitor.

## References
* megathread: #8888
* Similar, but not the same: #10274

## PR Checklist
* [x] Closes #10182
* [x] I work here
* [ ] Tests added/passed
* [n/a] Requires documentation to be updated

## Detailed Description of the Pull Request / Additional comments

We'll probably need to do this in a few more places, but I'm breaking PRs into small chunks for easier reviews.

## Validation Steps Performed

Summoned the window to a bunch of different resolutions. Where it would use the wrong size before, it no longer does.
@ghost
Copy link

ghost commented Aug 31, 2021

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

Handy links:

@ghost
Copy link

ghost commented Aug 31, 2021

🎉Windows Terminal Preview v1.11.2421.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-UserInterface Issues pertaining to the user interface of the Console or Terminal 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-2 A description (P2) Product-Terminal The new Windows Terminal.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Wrong size on Quake mode with hetero sized screens

3 participants