[PR #15178] Fix reordering tabs mysteriously shuffling the actual backing tabs #30475

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

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

State: closed
Merged: Yes


TL;DR: we stopped getting TabView.TabItemsChanged. This meant that the tab view would change its apparent order, but we wouldn't change the backing tab order.

I'm fixing this by grabbing the index of the tab that starts the drag, and the index of the tab view item at the end of the drag, and using that to reorder our backing list.

Closes #15121

Upstream https://github.com/microsoft/microsoft-ui-xaml/issues/8388

Regressed in #15078 - I'm pretty confident about this, since I've got a 1.18.931 build of the Terminal with tear-out, but not MUX 2.8.

**Original Pull Request:** https://github.com/microsoft/terminal/pull/15178 **State:** closed **Merged:** Yes --- TL;DR: we stopped getting `TabView.TabItemsChanged`. This meant that the tab view would change its apparent order, but we wouldn't change the backing tab order. I'm fixing this by grabbing the index of the tab that starts the drag, and the index of the tab view item at the end of the drag, and using that to reorder our backing list. Closes #15121 Upstream https://github.com/microsoft/microsoft-ui-xaml/issues/8388 Regressed in #15078 - I'm pretty confident about this, since I've got a 1.18.931 build of the Terminal with tear-out, but not MUX 2.8.
claunia added the pull-request label 2026-01-31 09:41:01 +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#30475