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

Fix ScrollRect to DECCRA translation #17849

Merged
merged 1 commit into from
Sep 4, 2024

Conversation

lhecker
Copy link
Member

@lhecker lhecker commented Sep 3, 2024

By translating the clip rectangle into a source-relative coordinate
space we can calculate the intersection that must be copied
much much more easily. I should've done that from the start.

Closes #17801

Validation Steps Performed

@microsoft-github-policy-service microsoft-github-policy-service bot added Issue-Bug It either shouldn't be doing this or needs an investigation. Area-VT Virtual Terminal sequence support Product-Conpty For console issues specifically related to conpty labels Sep 3, 2024
@lhecker lhecker force-pushed the dev/lhecker/17801-scroll-rect-fixup branch from 76d8507 to 90846ce Compare September 3, 2024 14:15
@DHowett DHowett merged commit d2c3cfd into main Sep 4, 2024
20 checks passed
@DHowett DHowett deleted the dev/lhecker/17801-scroll-rect-fixup branch September 4, 2024 23:06
DHowett pushed a commit that referenced this pull request Sep 6, 2024
By translating the clip rectangle into a source-relative coordinate
space we can calculate the intersection that must be copied
much much more easily. I should've done that from the start.

Closes #17801

## Validation Steps Performed
* Test code provided in #17801

(cherry picked from commit d2c3cfd)
Service-Card-Id: PVTI_lADOAF3p4s4AmhmQzgSa7nQ
Service-Version: 1.22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-VT Virtual Terminal sequence support Issue-Bug It either shouldn't be doing this or needs an investigation. Product-Conpty For console issues specifically related to conpty
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Some ScrollConsoleScreenBuffer directions have stopped working correctly
3 participants