Repository navigation
fix(tui): enable mouse + focus left nav pane by default - #921
Conversation
Mouse was off in tui-tmux.conf so clicking the left nav pane didn't give it focus. Also select left pane after TUI setup so keyboard input goes to OpenTUI nav by default (not the right terminal pane).
|
Warning You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again! |
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 379b25ef9d
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| # Mouse off (apps handle their own via passthrough) | ||
| set -g mouse off | ||
| # Mouse on — needed for clicking between left nav and right terminal panes | ||
| set -g mouse on |
There was a problem hiding this comment.
Remove conflicting mouse-off override
Setting mouse on in scripts/tmux/tui-tmux.conf is overridden during startup by applyTuiStyle() (src/term-commands/serve.ts, set-option -t ${TUI_SESSION} mouse off), so the advertised click-to-switch behavior is still disabled in the normal serve path. This makes the commit’s mouse fix ineffective unless the runtime style command fails, so the runtime setting should be updated to keep mouse enabled.
Useful? React with 👍 / 👎.
Mouse was off, clicking left nav didn't work. Left pane wasn't focused so j/k went to terminal.