ESC from the Tab ContextMenu should return focus to the Terminal #7949

Closed
opened 2026-01-31 01:16:36 +00:00 by claunia · 5 comments
Owner

Originally created by @zadjii-msft on GitHub (May 5, 2020).

Repro steps

  1. Right click a tab
  2. Hit esc

Expected

  1. Terminal gets focus again

Actual

  1. Focus lands on the tab itself
    image

regressed in #3789

Also, fixing this might just magically fix #3609.

Originally created by @zadjii-msft on GitHub (May 5, 2020). ### Repro steps 1. Right click a tab 2. Hit <kbd>esc</kbd> #### Expected 3. Terminal gets focus again #### Actual 3. Focus lands on the tab itself ![image](https://user-images.githubusercontent.com/18356694/81071815-d5c0fa80-8eaa-11ea-83f5-8339a2fcb2b4.png) regressed in #3789 Also, fixing this might just magically fix #3609.
Author
Owner

@zadjii-msft commented on GitHub (May 8, 2020):

Note from an offline discussion: maybe we could prevent the right click on the tab from focusing the TabViewItem, and then focus would just return to the Terminal? 🤞

@zadjii-msft commented on GitHub (May 8, 2020): Note from an offline discussion: maybe we could prevent the right click on the tab from focusing the TabViewItem, and then focus would just return to the Terminal? 🤞
Author
Owner

@zadjii-msft commented on GitHub (May 6, 2021):

Huh. This was fixed... some time? TabHeaderControl::_CloseRenameBox -> _RenameEndedHandlers -> handled in newTabImpl->TabRenamerDeactivated (lambda in TerminalPage)

Looks like it was originally in #9162, which was in v1.6.10571.0/Preview v1.7.572.0

@zadjii-msft commented on GitHub (May 6, 2021): Huh. This was fixed... some time? `TabHeaderControl::_CloseRenameBox` -> `_RenameEndedHandlers` -> handled in `newTabImpl->TabRenamerDeactivated` (lambda in `TerminalPage`) Looks like it was originally in #9162, which was in `v1.6.10571.0`/`Preview v1.7.572.0`
Author
Owner

@zadjii-msft commented on GitHub (May 6, 2021):

Wait no what? I'm crazy, had the wrong thread open this whole time. Ignore me.

@zadjii-msft commented on GitHub (May 6, 2021): Wait no what? I'm crazy, had the wrong thread open this whole time. Ignore me.
Author
Owner

@ghost commented on GitHub (May 25, 2021):

:tada:This issue was addressed in #10048, which has now been successfully released as Windows Terminal v1.8.1444.0.🎉

Handy links:

@ghost commented on GitHub (May 25, 2021): :tada:This issue was addressed in #10048, which has now been successfully released as `Windows Terminal v1.8.1444.0`.:tada: Handy links: * [Release Notes](https://github.com/microsoft/terminal/releases/tag/v1.8.1444.0) * [Store Download](https://www.microsoft.com/store/apps/9n8g5rfz9xk3?cid=storebadge&ocid=badge)
Author
Owner

@ghost commented on GitHub (May 25, 2021):

:tada:This issue was addressed in #10048, which has now been successfully released as Windows Terminal Preview v1.9.1445.0.🎉

Handy links:

@ghost commented on GitHub (May 25, 2021): :tada:This issue was addressed in #10048, which has now been successfully released as `Windows Terminal Preview v1.9.1445.0`.:tada: Handy links: * [Release Notes](https://github.com/microsoft/terminal/releases/tag/v1.9.1445.0) * [Store Download](https://www.microsoft.com/store/apps/9n8g5rfz9xk3?cid=storebadge&ocid=badge)
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#7949