Skip to content

fix: sidebar interaction blocked by canvas area#21

Merged
190km merged 1 commit intomainfrom
fix/sidebar-click-through
Mar 29, 2026
Merged

fix: sidebar interaction blocked by canvas area#21
190km merged 1 commit intomainfrom
fix/sidebar-click-through

Conversation

@190km
Copy link
Copy Markdown
Owner

@190km 190km commented Mar 29, 2026

Summary

  • The canvas_content Area in app.rs was positioned at Pos2::ZERO (screen coordinate origin), causing it to span the entire screen including the sidebar region
  • Since this Area is rendered at Order::Middle (above the sidebar panel at the base layer), it intercepted pointer events — such as right-clicks — meant for sidebar items whenever terminal panels on the canvas overlapped the sidebar area
  • Changed fixed_pos(Pos2::ZERO) to fixed_pos(canvas_rect.min) so the Area only covers the canvas region, allowing all sidebar interactions to work correctly

Test plan

  • Open Void with multiple terminal panels, positioning at least one so it overlaps with or is near the sidebar area
  • Right-click on sidebar items (workspace entries, terminal list items) — context menus should appear
  • Left-click sidebar items to switch workspaces/focus terminals — should work reliably
  • Verify canvas interactions (pan, zoom, drag panels, resize panels) still work correctly
  • Verify minimap overlay still renders and functions properly

🤖 Generated with Claude Code

The canvas_content Area was positioned at Pos2::ZERO (screen origin),
causing it to cover the entire screen including the sidebar region.
Since this Area renders at Order::Middle (above the sidebar panel),
it intercepted pointer events meant for sidebar items when terminal
panels overlapped the sidebar area.

Moving the Area origin to canvas_rect.min restricts it to the canvas
region only, allowing sidebar right-clicks and other interactions to
work correctly.
@190km 190km merged commit b027bbe into main Mar 29, 2026
8 checks passed
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.

1 participant