Skip to content

fix(tui): let mouse capture keep history arrows on Windows#2103

Open
Hmbown wants to merge 2 commits into
mainfrom
codex/issue-1720-windows-arrow-history
Open

fix(tui): let mouse capture keep history arrows on Windows#2103
Hmbown wants to merge 2 commits into
mainfrom
codex/issue-1720-windows-arrow-history

Conversation

@Hmbown
Copy link
Copy Markdown
Owner

@Hmbown Hmbown commented May 25, 2026

Summary

  • removes the blanket Windows override from the empty-composer arrow default
  • keeps the scroll fallback when mouse capture is off, including Windows conhost-style terminals
  • updates the config comment and regression coverage for both Windows default cases

Fixes #1720

Credit: harvests the focused behavior from Lee-take's #1866 and Paulo's Windows terminal matrix in the issue thread.

Validation

  • cargo fmt --all -- --check
  • CARGO_TARGET_DIR=/Volumes/VIXinSSD/whalebro/codewhale-target/issue-1720 cargo test -p codewhale-tui composer_arrows_scroll_default

Copilot AI review requested due to automatic review settings May 25, 2026 05:54
@gemini-code-assist
Copy link
Copy Markdown
Contributor

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

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.

Empty composer: Up/Down scrolls transcript instead of navigating history on Windows

2 participants