Skip to content

Make terminal list click focus the tile instead of just peeking - #83

Open
piske-alex wants to merge 1 commit into
collabs-inc:mainfrom
piske-alex:feat/terminal-list-click-focus
Open

Make terminal list click focus the tile instead of just peeking#83
piske-alex wants to merge 1 commit into
collabs-inc:mainfrom
piske-alex:feat/terminal-list-click-focus

Conversation

@piske-alex

Copy link
Copy Markdown
Contributor

Clicking a terminal entry in the sidebar now pans the canvas to the tile AND focuses it (bringing it to front with keyboard input ready). Previously clicking only panned without focusing, requiring two extra clicks to interact with the terminal.

  • terminal-list App.tsx: click calls focusTile() instead of peekTile()
  • renderer.js: add terminal-list:focus-tile handler that pans + focuses
  • renderer.js: add terminal-list:blur handler for Escape key
  • tile-manager.js: simulate a click in terminal webview center on programmatic focus so xterm grabs keyboard input immediately

Arrow keys still peek (pan without focus) for quick browsing. Enter focuses the peeked tile (existing behavior).

Clicking a terminal entry in the sidebar now pans the canvas to
the tile AND focuses it (bringing it to front with keyboard input
ready). Previously clicking only panned without focusing, requiring
an extra click to interact with the terminal.

- terminal-list App.tsx: click calls focusTile() instead of peekTile()
- renderer.js: add terminal-list:focus-tile handler that pans + focuses
- renderer.js: add terminal-list:blur handler for Escape key
- tile-manager.js: simulate a click in terminal webview center on
  programmatic focus so xterm grabs keyboard input immediately

Arrow keys still peek (pan without focus) for quick browsing.
Enter focuses the peeked tile (existing behavior).

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant