[PR #12667] AtlasEngine: Fix ConstBuffer invalidation for background color changes #29176

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

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

State: closed
Merged: Yes


The bg != _r.backgroundColor invalidation check wasn't symmetric with
us setting _r.backgroundColor to bg | _api.backgroundOpaqueMixin.
Due to this, when the backgroundOpaqueMixin changed,
we didn't always update the const buffer appropriately.

PR Checklist

Validation Steps Performed

  • Enable window transparency (backgroundOpaqueMixin == 0x00000000)
  • Maximize window (ensure we have gutters in the first place)
  • Disable window transparency (backgroundOpaqueMixin == 0xff000000)
  • Gutters are filled
**Original Pull Request:** https://github.com/microsoft/terminal/pull/12667 **State:** closed **Merged:** Yes --- The `bg != _r.backgroundColor` invalidation check wasn't symmetric with us setting `_r.backgroundColor` to `bg | _api.backgroundOpaqueMixin`. Due to this, when the `backgroundOpaqueMixin` changed, we didn't always update the const buffer appropriately. ## PR Checklist * [x] Closes #11773 * [x] I work here * [x] Tests added/passed ## Validation Steps Performed * Enable window transparency (`backgroundOpaqueMixin == 0x00000000`) * Maximize window (ensure we have gutters in the first place) * Disable window transparency (`backgroundOpaqueMixin == 0xff000000`) * Gutters are filled ✅
claunia added the pull-request label 2026-01-31 09:33:16 +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#29176