charles/loom
1
0
Fork 0
Code Issues 4 Pull requests Projects Releases Packages Wiki Activity Actions

feat(tui): mouse support — click, scroll, overlay dismiss #116

Merged
claude-desktop merged 2 commits from tui/mouse-support-112 into main 2026-04-12 18:45:03 +00:00
Conversation 0 Commits 2 Files changed 11 +413 -16
claude-desktop commented 2026-04-12 18:30:53 +00:00
Collaborator
Copy link

Summary

  • Plumb Event::Mouse through the full dispatch chain (event stream → overlays → screens)
  • Hit-test helper hit_test(rect, column, row) for click target detection
  • Sidebar click navigation: click a sidebar row to switch screens
  • Click outside an overlay to close it; clicks inside are consumed
  • Scroll wheel support in gallery, entities, presets, model browser
  • mouse_enabled config guard — tui.toml [tui] mouse_enabled = false disables capture
  • overlay_rect() trait method for hit-testing overlay bounds
  • 5 new integration tests + 3 unit tests for hit_test

Closes charles/loom#112

Test plan

  • cargo test -p loom-tui — all tests pass
  • cargo clippy -p loom-tui -- -D warnings — clean
  • Manual: click sidebar items to navigate between screens
  • Manual: scroll wheel in gallery grid moves selection
  • Manual: open help (?), click outside to dismiss
  • Manual: set mouse_enabled = false in tui.toml, verify mouse has no effect

🤖 Generated with Claude Code

## Summary - Plumb `Event::Mouse` through the full dispatch chain (event stream → overlays → screens) - Hit-test helper `hit_test(rect, column, row)` for click target detection - Sidebar click navigation: click a sidebar row to switch screens - Click outside an overlay to close it; clicks inside are consumed - Scroll wheel support in gallery, entities, presets, model browser - `mouse_enabled` config guard — `tui.toml` `[tui] mouse_enabled = false` disables capture - `overlay_rect()` trait method for hit-testing overlay bounds - 5 new integration tests + 3 unit tests for hit_test Closes charles/loom#112 ## Test plan - [x] `cargo test -p loom-tui` — all tests pass - [x] `cargo clippy -p loom-tui -- -D warnings` — clean - [ ] Manual: click sidebar items to navigate between screens - [ ] Manual: scroll wheel in gallery grid moves selection - [ ] Manual: open help (`?`), click outside to dismiss - [ ] Manual: set `mouse_enabled = false` in tui.toml, verify mouse has no effect 🤖 Generated with [Claude Code](https://claude.com/claude-code)
Plumb mouse events through the full dispatch chain:

- hit_test() helper for rect-based mouse coordinate checks
- Sidebar click navigation: left-click on sidebar items navigates
- Gallery/entities/presets/model-browser scroll wheel moves selection
- Overlay click-outside-to-close with per-overlay rect tracking
- All overlays swallow mouse events inside their bounds
- Config mouse_enabled field (default true) gates mouse capture
- terminal::init_with_mouse() conditionally enables EnableMouseCapture

Closes charles/loom#112

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
claude-desktop changed target branch from tui/test-harness-fixes to main 2026-04-12 18:40:11 +00:00
charles force-pushed tui/mouse-support-112 from e3acf9e445 to f0fd06d622
All checks were successful
QA / qa (pull_request) Successful in 34m11s
Details
2026-04-12 18:44:57 +00:00 Compare
claude-desktop deleted branch tui/mouse-support-112 2026-04-12 18:45:04 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
area:agents
Agent types, pool, config

  
area:ai
AI features (prompt enhancement, tags, face detection, etc.)

  
area:config
Configuration files & loading

  
area:dashboard
Dashboard UI + observability

  
area:design
UI/UX mockups — routes to designer

  
area:design-review
Design review — routes to design-reviewer

  
area:devtools
Log / debug overlays

  
area:entities
Entities screen

  
area:gallery
Gallery screen

  
area:generate
Generate screen

  
area:image
Image rendering subsystem (kitty/sixel/chafa)

  
area:infra
Scaffolding, CI, packaging

  
area:meta
Tracking, scaffolding, setup

  
area:model-browser
Model browser screen

  
area:navigation
Sidebar, status bar, palette, keybinds

  
area:presets
Presets screen

  
area:security
Security — routes to reviewer-security (opus)

  
area:sessions
Session store, SDK resume

  
area:settings
Settings screen

  
area:sharing
Sharing overlay

  
area:test
Automated tests

  
area:ux
UX polish, notifications, error handling

  
area:webhook
Webhook routing + handlers

  
area:workdir
Clone cache, worktrees, git identity

  
type:bug
Bug

  
type:chore
Chore

  
type:meta
Tracking / decisions, not implementation

  
type:user-story
User story ticket

No labels
area:agents
area:ai
area:config
area:dashboard
area:design
area:design-review
area:devtools
area:entities
area:gallery
area:generate
area:image
area:infra
area:meta
area:model-browser
area:navigation
area:presets
area:security
area:sessions
area:settings
area:sharing
area:test
area:ux
area:webhook
area:workdir
type:bug
type:chore
type:meta
type:user-story
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
charles/loom!116
No description provided.