Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

AtlasEngine: Stop resizing buffers on scroll #13100

Merged
1 commit merged into from May 17, 2022
Merged

Conversation

lhecker
Copy link
Member

@lhecker lhecker commented May 14, 2022

This regressed in ad2358d.
We're interested in the size of the viewport only, but it can shift up/down
during scrolling. In these situations we shouldn't resize our buffers of course.

Validation Steps Performed

  • Scroll
  • Not setting ApiInvalidations::Size

@lhecker lhecker added Issue-Bug It either shouldn't be doing this or needs an investigation. Priority-2 A description (P2) Area-AtlasEngine labels May 14, 2022
@lhecker lhecker added this to the Terminal v1.14 milestone May 14, 2022
@lhecker lhecker added this to To Cherry Pick in 1.14 Servicing Pipeline via automation May 14, 2022
@lhecker lhecker removed this from the Terminal v1.14 milestone May 14, 2022
@lhecker lhecker removed this from To Cherry Pick in 1.14 Servicing Pipeline May 14, 2022
@lhecker
Copy link
Member Author

lhecker commented May 14, 2022

Turns out ad2358d isn't in 1.14 (I think?) so it doesn't need to be backported.

@DHowett
Copy link
Member

DHowett commented May 14, 2022

I think that commit is in 1.14. We should tag this one up.

@DHowett DHowett added this to To Cherry Pick in 1.14 Servicing Pipeline via automation May 16, 2022
@zadjii-msft zadjii-msft added this to the Terminal v1.14 milestone May 16, 2022
@lhecker lhecker added the AutoMerge Marked for automatic merge by the bot when requirements are met label May 16, 2022
@ghost
Copy link

ghost commented May 16, 2022

Hello @lhecker!

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 fa6b066 into main May 17, 2022
@ghost ghost deleted the dev/lhecker/atlas-engine-fixup branch May 17, 2022 00:07
@carlos-zamora carlos-zamora moved this from To Cherry Pick to Cherry Picked in 1.14 Servicing Pipeline May 17, 2022
carlos-zamora pushed a commit that referenced this pull request May 17, 2022
This regressed in ad2358d.
We're interested in the size of the viewport only, but it can shift up/down
during scrolling. In these situations we shouldn't resize our buffers of course.

## Validation Steps Performed
* Scroll
* Not setting `ApiInvalidations::Size` ✅

(cherry picked from commit fa6b066)
Service-Card-Id: 82005202
Service-Version: 1.14
@ghost
Copy link

ghost commented May 24, 2022

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

Handy links:

@DHowett DHowett moved this from Cherry Picked to Validated in 1.14 Servicing Pipeline May 25, 2022
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-AtlasEngine 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-2 A description (P2)
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

4 participants