Skip to content

Replace scroll-based queue drawer expansion with deliberate drag gesture#759

Open
marcodejongh wants to merge 1 commit intomainfrom
claude/fix-queue-drawer-drag-3K13W
Open

Replace scroll-based queue drawer expansion with deliberate drag gesture#759
marcodejongh wants to merge 1 commit intomainfrom
claude/fix-queue-drawer-drag-3K13W

Conversation

@marcodejongh
Copy link
Owner

The queue drawer height (60% → 100%) was expanding automatically when the user scrolled the queue list content. Now it only resizes when the user deliberately drags the header/drag handle up or down.

  • Removed scroll event listener that auto-expanded the drawer
  • Added custom drag header with touch event handlers for resize
  • Drag up on header: expand to 100%
  • Drag down from 100%: collapse to 60%
  • Drag down from 60%: close the drawer
  • Disabled MUI SwipeableDrawer swipe gestures on queue drawer to prevent conflicts with the custom drag handler

https://claude.ai/code/session_01KBC5LtVtKaZihb3oMcHQp5

The queue drawer height (60% → 100%) was expanding automatically when the
user scrolled the queue list content. Now it only resizes when the user
deliberately drags the header/drag handle up or down.

- Removed scroll event listener that auto-expanded the drawer
- Added custom drag header with touch event handlers for resize
- Drag up on header: expand to 100%
- Drag down from 100%: collapse to 60%
- Drag down from 60%: close the drawer
- Disabled MUI SwipeableDrawer swipe gestures on queue drawer to prevent
  conflicts with the custom drag handler

https://claude.ai/code/session_01KBC5LtVtKaZihb3oMcHQp5
@vercel
Copy link

vercel bot commented Feb 8, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
boardsesh Building Building Feb 8, 2026 1:11pm

Request Review

@claude
Copy link

claude bot commented Feb 8, 2026

Claude Review

Ready to merge - Minor issues noted below, but nothing blocking.

Issues

  1. Missing desktop support (play-view-drawer.tsx:126-159): The drag handlers only use touch events (onTouchStart, onTouchMove, onTouchEnd). Desktop users cannot resize the drawer by dragging with a mouse. Consider adding corresponding mouse event handlers.

  2. Redundant color prop (play-view-drawer.tsx:440-441): color={showHistory ? 'default' : 'default'} - both branches return the same value, making the ternary pointless.

  3. No tests: This is a UX behavior change (scroll-based expansion → drag gesture) with no accompanying tests.

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