fix(preview): enable text element dragging in preview panel#693
fix(preview): enable text element dragging in preview panel#693nino-chavez wants to merge 1 commit intoOpenCut-app:mainfrom
Conversation
Text elements are now draggable in the video preview: - Added invisible DOM overlay layer for text elements - Overlay captures mouse events while canvas renders visuals - Uses same positioning formula as timeline-renderer.ts - Drag state already existed, just needed DOM element to trigger it Fixes OpenCut-app#614 Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
|
@nino-chavez is attempting to deploy a commit to the OpenCut OSS Team on Vercel. A member of the Team first needs to authorize it. |
👷 Deploy request for appcut pending review.Visit the deploys page to approve it
|
|
Warning Rate limit exceeded
⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Summary
Fixes the issue where text elements could not be dragged/repositioned in the video preview panel.
Root cause: After the preview was refactored to canvas-based rendering for performance, text elements were only drawn on canvas with no DOM elements to capture mouse events. The drag handlers existed but had nothing to attach to.
Solution: Added an invisible DOM overlay layer that:
Related Issue
Fixes #614
Technical Details
opacity: 0) - canvas still renders the actual textposX = previewWidth/2 + text.x * scaleXTest plan
🤖 Generated with Claude Code