Code Health: Cleanup manual StartBringIntoView once fixed in WinUI #12125

Open
opened 2026-01-31 03:06:55 +00:00 by claunia · 2 comments
Owner

Originally created by @Don-Vito on GitHub (Jan 20, 2021).

As a part of a workaround for https://github.com/microsoft/microsoft-ui-xaml/issues/3945 we have introduced a manual call to StartBringIntoView (https://github.com/microsoft/terminal/pull/8832).

We would like to cleanup this code, once the fix is introduced in the TabView.
There is already a PR for the upstream issue here: https://github.com/microsoft/microsoft-ui-xaml/pull/3966.

Originally created by @Don-Vito on GitHub (Jan 20, 2021). As a part of a workaround for https://github.com/microsoft/microsoft-ui-xaml/issues/3945 we have introduced a manual call to `StartBringIntoView` (https://github.com/microsoft/terminal/pull/8832). We would like to cleanup this code, once the fix is introduced in the TabView. There is already a PR for the upstream issue here: https://github.com/microsoft/microsoft-ui-xaml/pull/3966.
claunia added the Help WantedIssue-TaskProduct-TerminalArea-CodeHealth labels 2026-01-31 03:06:55 +00:00
Author
Owner

@zadjii-msft commented on GitHub (Jan 20, 2021):

Yep. Thanks for filing this (and fixing it)!

@zadjii-msft commented on GitHub (Jan 20, 2021): Yep. Thanks for filing this (and fixing it)!
Author
Owner

@zadjii-msft commented on GitHub (Apr 1, 2024):

Huh, the upstream https://github.com/microsoft/microsoft-ui-xaml/issues/3945 was fixed. We can probably fix this now?

@zadjii-msft commented on GitHub (Apr 1, 2024): Huh, the upstream https://github.com/microsoft/microsoft-ui-xaml/issues/3945 was fixed. We can probably fix this now?
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#12125