[PR #15425] AtlasEngine: Improve scroll and swap chain invalidation #30632

Open
opened 2026-01-31 09:41:58 +00:00 by claunia · 0 comments
Owner

Original Pull Request: https://github.com/microsoft/terminal/pull/15425

State: closed
Merged: Yes


_p.MarkAllAsDirty() sets _p.scrollOffset to 0, so we need to use
that instead of _api.scrollOffset when getting the offset.
Additionally, the previous code failed to release the swap chain
when recreating the backend, which is technically not correct.
I'm not sure to what issues this might have led, as it didn't had any
negative effects on my PC, but it's definitely not according to spec.

Validation Steps Performed

Difficult to test but it seems alright.

**Original Pull Request:** https://github.com/microsoft/terminal/pull/15425 **State:** closed **Merged:** Yes --- `_p.MarkAllAsDirty()` sets `_p.scrollOffset` to 0, so we need to use that instead of `_api.scrollOffset` when getting the offset. Additionally, the previous code failed to release the swap chain when recreating the backend, which is technically not correct. I'm not sure to what issues this might have led, as it didn't had any negative effects on my PC, but it's definitely not according to spec. ## Validation Steps Performed Difficult to test but it seems alright.
claunia added the pull-request label 2026-01-31 09:41:58 +00:00
Sign in to join this conversation.
No Label pull-request
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#30632