Skip to content

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

Merged
Hmbown merged 2 commits into
mainfrom
codex/issue-1720-windows-arrow-history
May 26, 2026
Merged

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

Conversation

@Hmbown

@Hmbown Hmbown commented May 25, 2026

Copy link
Copy Markdown
Owner

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!

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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.

@Hmbown Hmbown merged commit b3c50e0 into main May 26, 2026
12 checks passed
@Hmbown Hmbown deleted the codex/issue-1720-windows-arrow-history branch June 14, 2026 19:35
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