Skip to content

fix(desktop): focus the tab restored by undo-close (⌘⇧T) - #74640

Merged
OutThisLife merged 1 commit into
mainfrom
bb/undo-close-tab-focus
Jul 30, 2026
Merged

fix(desktop): focus the tab restored by undo-close (⌘⇧T)#74640
OutThisLife merged 1 commit into
mainfrom
bb/undo-close-tab-focus

Conversation

@OutThisLife

Copy link
Copy Markdown
Collaborator

Summary

  • ⌘⇧T only put the closed tile back in $sessionTiles. Pane adoption is silent, so the tab came back behind the still-fronted workspace.
  • reopenLastClosedTile now calls focusOpenSession after open so the restored tab is fronted and its zone is the focus target again.

Test plan

  • Open a session tab, ⌘W it, ⌘⇧T — the restored tab is active (not buried behind main)
  • Titlebar / statusbar / next ⌘W target follow the restored tab
  • vitest run src/store/session-states.test.ts (reopen-focus case)

Adoption alone is silent, so reopenLastClosedTile only restored placement
and left the tab behind the still-fronted workspace. Focus it after open.
@OutThisLife
OutThisLife enabled auto-merge July 30, 2026 06:15
@alt-glitch alt-glitch added type/bug Something isn't working P3 Low — cosmetic, nice to have comp/desktop Electron desktop app (apps/desktop/*) area/sessions Session lifecycle, resume, persistence, history labels Jul 30, 2026
@github-actions

github-actions Bot commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

૮ >ﻌ< ა ci review

ran on 7556c51

ℹ️ Info

Desktop E2E visual evidence · View test artifacts · View job

1 visual diff.

inline evidence is publishing...

@OutThisLife
OutThisLife merged commit 05f5df6 into main Jul 30, 2026
37 checks passed
@OutThisLife
OutThisLife deleted the bb/undo-close-tab-focus branch July 30, 2026 07:06
randlee pushed a commit to randlee/hermes-agent that referenced this pull request Aug 11, 2026
…-tab-focus

fix(desktop): focus the tab restored by undo-close (⌘⇧T)
33hodl pushed a commit to 33hodl/hermes-agent that referenced this pull request Aug 12, 2026
…-tab-focus

fix(desktop): focus the tab restored by undo-close (⌘⇧T)
prmartinow pushed a commit to prmartinow/hermes-agent that referenced this pull request Aug 26, 2026
…-tab-focus

fix(desktop): focus the tab restored by undo-close (⌘⇧T)
melon-xf added a commit to melon-xf/hermes-agent that referenced this pull request Sep 3, 2026
…-tab-focus

fix(desktop): focus the tab restored by undo-close (⌘⇧T)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/sessions Session lifecycle, resume, persistence, history comp/desktop Electron desktop app (apps/desktop/*) P3 Low — cosmetic, nice to have type/bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants