Skip to content

fix(ui): focus newly opened terminal sessions - #56

Closed
ChrisLauinger77 wants to merge 1 commit into
cheat2001:mainfrom
ChrisLauinger77:codex/fix-terminal-focus
Closed

ChrisLauinger77 wants to merge 1 commit into
cheat2001:mainfrom
ChrisLauinger77:codex/fix-terminal-focus

Conversation

@ChrisLauinger77

Copy link
Copy Markdown
Contributor

What & why

Clicking the terminal button for the first time opened the panel without keyboard focus, so typing required another click inside the terminal. Focus was only applied when reattaching an existing session. Apply it after both setup paths so newly opened and reopened terminals accept typing immediately. Extend the terminal flow test to check focus, echoed keystrokes, and retained input after reopening.

Development StrictMode masked the missing call by running the attachment effect twice. The production preview reproduced the failure before the fix and passes afterward.

Screenshots

This changes keyboard focus without changing the visual layout. The browser regression verifies focus and typed input after opening and reopening.

Validation

  • Workspace typecheck passed.
  • All 238 unit tests passed.
  • Desktop production build passed.
  • Production browser checks passed for terminal focus/reopening/context-menu actions and font settings (2 tests, installed Chromium).
  • The focus check failed on the production build before the fix.

Checklist

  • pnpm typecheck && pnpm test pass
  • CLAUDE.md documents the focus behavior and StrictMode testing pitfall
  • Commits carry no trailers from coding tools

Rust, engine integration, undo, and color checks are not applicable to this focus-only frontend change.

@cheat2001

Copy link
Copy Markdown
Owner

Landed on main as b2f9ebb under your name. The changelog had moved underneath it, so I squashed it locally, which is why GitHub shows closed rather than merged. Thanks Chris, it goes out with the next release.

@cheat2001 cheat2001 closed this Oct 7, 2026
@ChrisLauinger77
ChrisLauinger77 deleted the codex/fix-terminal-focus branch October 7, 2026 17:27
@ChrisLauinger77

ChrisLauinger77 commented Oct 7, 2026 •

Copy link
Copy Markdown
Contributor Author

Thx man, I really like the repo sorting on the start page and the color option.
What do you think about allow pull the complete repos from 1 category of the main page feature ?

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.

2 participants