Skip to content

tui: make column options checkboxes clickable with mouse#507

Merged
wesm merged 2 commits intomainfrom
mouse-checkbox
Mar 12, 2026
Merged

tui: make column options checkboxes clickable with mouse#507
wesm merged 2 commits intomainfrom
mouse-checkbox

Conversation

@mariusvniekerk
Copy link
Collaborator

Summary

  • Add mouse click handling for the column options modal so checkbox items can be toggled by clicking
  • Add mouse wheel scroll support for navigating the column options list
  • Extract toggle logic into toggleColumnOption() helper shared by keyboard and mouse handlers
  • Fix separator blank line click incorrectly toggling the adjacent option (now a no-op)

🤖 Generated with Claude Code

mariusvniekerk and others added 2 commits March 12, 2026 12:34
Add mouse click handling for the column options modal so checkbox items
can be toggled by clicking. Also adds mouse wheel scroll support in the
modal. Extracts toggle logic into toggleColumnOption() helper to share
between keyboard and mouse handlers.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
… modal

Clicking the blank separator line between column toggles and sentinel
options was decrementing the row index and toggling the last column
option above it. Now the separator row is a no-op.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@roborev-ci
Copy link

roborev-ci bot commented Mar 12, 2026

roborev: Combined Review (2c3535d)

Verdict: Clean

All agents
agree the code is clean. No issues were found.


Synthesized from 3 reviews (agents: codex, gemini | types: default, security)

@wesm wesm merged commit 3af6043 into main Mar 12, 2026
8 checks passed
@wesm wesm deleted the mouse-checkbox branch March 12, 2026 19:06
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