[PR #17174] Don't always focus pane content on Tapped, if the pane is already focused #31144

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

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

State: closed
Merged: Yes


You'll never believe this. Clicking on the dropdown button on a ComboBox doesn't set e.Tapped = true. It bubbles up, and lands in our Pane's Border's tapped handler. And in there, we yeet focus to the first content. We end up stealing focus from the combobox, and then the combobox doesn't actually open its dropdown.

So yea we can just fix that. Easy enough.

Closes #17062

**Original Pull Request:** https://github.com/microsoft/terminal/pull/17174 **State:** closed **Merged:** Yes --- You'll never believe this. Clicking on the dropdown button on a ComboBox doesn't set `e.Tapped = true`. It bubbles up, and lands in our `Pane`'s `Border`'s tapped handler. And in there, we yeet focus to the first content. We end up stealing focus from the combobox, and then the combobox doesn't actually open its dropdown. So yea we can just fix that. Easy enough. Closes #17062
claunia added the pull-request label 2026-01-31 09:45:23 +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#31144