[PR #11369] Allow the entire Tab to be hit testable again #28531

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

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

State: closed
Merged: Yes


DESPITE the fact that there's a Background() API that we
could just call like:

  TabViewItem().Background(deselectedTabBrush);

We actually can't, because it will make the part of the tab that
doesn't contain the text totally transparent to hit tests. So we
actually do still need to set TabViewItemHeaderBackground manually.

**Original Pull Request:** https://github.com/microsoft/terminal/pull/11369 **State:** closed **Merged:** Yes --- DESPITE the fact that there's a `Background()` API that we could just call like: ```c++ TabViewItem().Background(deselectedTabBrush); ``` We actually can't, because it will make the part of the tab that doesn't contain the text totally transparent to hit tests. So we actually _do_ still need to set `TabViewItemHeaderBackground` manually. * Regressed in #11240 * Root cause up in https://github.com/microsoft/microsoft-ui-xaml/pull/3769 * [x] closes #11294
claunia added the pull-request label 2026-01-31 09:29:07 +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#28531