From 630250c23b1cd8ec608c95334ae2babf8e7a00af Mon Sep 17 00:00:00 2001 From: aashishparuvada Date: Mon, 23 Jun 2025 20:51:36 +0530 Subject: [PATCH] enabled-multiclip-selection --- apps/web/src/components/editor/timeline.tsx | 165 +++++++++++++++++--- apps/web/src/stores/timeline-store.ts | 36 +++-- 2 files changed, 168 insertions(+), 33 deletions(-) diff --git a/apps/web/src/components/editor/timeline.tsx b/apps/web/src/components/editor/timeline.tsx index a16284a..1120fbe 100644 --- a/apps/web/src/components/editor/timeline.tsx +++ b/apps/web/src/components/editor/timeline.tsx @@ -33,7 +33,7 @@ export function Timeline() { // Timeline shows all tracks (video, audio, effects) and their clips. // You can drag media here to add it to your project. // Clips can be trimmed, deleted, and moved. - const { tracks, addTrack, addClipToTrack, removeTrack, toggleTrackMute, removeClipFromTrack, moveClipToTrack, getTotalDuration, selectedClip, clearSelectedClip } = + const { tracks, addTrack, addClipToTrack, removeTrack, toggleTrackMute, removeClipFromTrack, moveClipToTrack, getTotalDuration, selectedClips, selectClip, deselectClip, clearSelectedClips } = useTimelineStore(); const { mediaItems, addMediaItem } = useMediaStore(); const { currentTime, duration, seek, setDuration, isPlaying, play, pause, toggle } = usePlaybackStore(); @@ -52,6 +52,16 @@ export function Timeline() { y: number; } | null>(null); + // Marquee selection state + const [marquee, setMarquee] = useState<{ + startX: number; + startY: number; + endX: number; + endY: number; + active: boolean; + additive: boolean; + } | null>(null); + // Update timeline duration when tracks change useEffect(() => { const totalDuration = getTotalDuration(); @@ -67,17 +77,99 @@ export function Timeline() { } }, [contextMenu]); - // Keyboard event for deleting selected clip + // Keyboard event for deleting selected clips useEffect(() => { const handleKeyDown = (e: KeyboardEvent) => { - if ((e.key === "Delete" || e.key === "Backspace") && selectedClip) { - removeClipFromTrack(selectedClip.trackId, selectedClip.clipId); - clearSelectedClip(); + if ((e.key === "Delete" || e.key === "Backspace") && selectedClips.length > 0) { + selectedClips.forEach(({ trackId, clipId }) => { + removeClipFromTrack(trackId, clipId); + }); + clearSelectedClips(); } }; window.addEventListener("keydown", handleKeyDown); return () => window.removeEventListener("keydown", handleKeyDown); - }, [selectedClip, removeClipFromTrack, clearSelectedClip]); + }, [selectedClips, removeClipFromTrack, clearSelectedClips]); + + // Mouse down on timeline background to start marquee + const handleTimelineMouseDown = (e: React.MouseEvent) => { + if (e.target === e.currentTarget && e.button === 0) { + setMarquee({ + startX: e.clientX, + startY: e.clientY, + endX: e.clientX, + endY: e.clientY, + active: true, + additive: e.metaKey || e.ctrlKey || e.shiftKey, + }); + } + }; + + // Mouse move to update marquee + useEffect(() => { + if (!marquee || !marquee.active) return; + const handleMouseMove = (e: MouseEvent) => { + setMarquee((prev) => prev && { ...prev, endX: e.clientX, endY: e.clientY }); + }; + const handleMouseUp = (e: MouseEvent) => { + setMarquee((prev) => prev && { ...prev, endX: e.clientX, endY: e.clientY, active: false }); + }; + window.addEventListener("mousemove", handleMouseMove); + window.addEventListener("mouseup", handleMouseUp); + return () => { + window.removeEventListener("mousemove", handleMouseMove); + window.removeEventListener("mouseup", handleMouseUp); + }; + }, [marquee]); + + // On marquee end, select clips in box + useEffect(() => { + if (!marquee || marquee.active) return; + // Calculate selection box in timeline coordinates + const timeline = timelineRef.current; + if (!timeline) return; + const rect = timeline.getBoundingClientRect(); + const x1 = Math.min(marquee.startX, marquee.endX) - rect.left; + const x2 = Math.max(marquee.startX, marquee.endX) - rect.left; + const y1 = Math.min(marquee.startY, marquee.endY) - rect.top; + const y2 = Math.max(marquee.startY, marquee.endY) - rect.top; + // Find all clips that intersect the box + let newSelection: { trackId: string; clipId: string }[] = []; + tracks.forEach((track, trackIdx) => { + track.clips.forEach((clip) => { + const effectiveDuration = clip.duration - clip.trimStart - clip.trimEnd; + const clipWidth = Math.max(80, effectiveDuration * 50 * zoomLevel); + const clipLeft = clip.startTime * 50 * zoomLevel; + const clipTop = trackIdx * 60; + const clipBottom = clipTop + 60; + const clipRight = clipLeft + clipWidth; + // Check intersection + if ( + x1 < clipRight && + x2 > clipLeft && + y1 < clipBottom && + y2 > clipTop + ) { + newSelection.push({ trackId: track.id, clipId: clip.id }); + } + }); + }); + if (newSelection.length > 0) { + if (marquee.additive) { + // Add to current selection + const current = new Set(selectedClips.map((c) => c.trackId + ":" + c.clipId)); + newSelection = [ + ...selectedClips, + ...newSelection.filter((c) => !current.has(c.trackId + ":" + c.clipId)), + ]; + } + clearSelectedClips(); + newSelection.forEach((c) => selectClip(c.trackId, c.clipId, true)); + } else if (!marquee.additive) { + clearSelectedClips(); + } + setMarquee(null); + }, [marquee, tracks, zoomLevel, selectedClips, selectClip, clearSelectedClips]); const handleDragEnter = (e: React.DragEvent) => { // When something is dragged over the timeline, show overlay @@ -185,17 +277,12 @@ export function Timeline() { } }; - const handleTimelineClick = (e: React.MouseEvent) => { - const timeline = timelineRef.current; - if (!timeline || duration === 0) return; - - const rect = timeline.getBoundingClientRect(); - const x = e.clientX - rect.left; - const timelineWidth = rect.width; - const visibleDuration = duration / zoomLevel; - const clickedTime = (x / timelineWidth) * visibleDuration; - - seek(Math.max(0, Math.min(duration, clickedTime))); + // Deselect all clips when clicking empty timeline area + const handleTimelineAreaClick = (e: React.MouseEvent) => { + // Only clear selection if the click target is the timeline background (not a child/clip) + if (e.target === e.currentTarget) { + clearSelectedClips(); + } }; const handleWheel = (e: React.WheelEvent) => { @@ -500,9 +587,33 @@ export function Timeline() { minHeight: tracks.length > 0 ? `${tracks.length * 60}px` : "200px", }} - onClick={handleTimelineClick} + onClick={handleTimelineAreaClick} + onMouseDown={handleTimelineMouseDown} onWheel={handleWheel} > + {/* Overlay for deselect/marquee */} +
+ {/* Marquee selection rectangle */} + {marquee && marquee.active && timelineRef.current && ( +
+ )} {tracks.length === 0 ? (
@@ -711,8 +822,9 @@ function TimelineTrackContent({ addClipToTrack, removeClipFromTrack, toggleTrackMute, - selectedClip, + selectedClips, selectClip, + deselectClip, } = useTimelineStore(); const { currentTime } = usePlaybackStore(); const [isDropping, setIsDropping] = useState(false); @@ -1274,10 +1386,9 @@ function TimelineTrackContent({ effectiveDuration * 50 * zoomLevel ); const clipLeft = clip.startTime * 50 * zoomLevel; - - // Correctly declare isSelected inside the map - const isSelected = selectedClip && selectedClip.trackId === track.id && selectedClip.clipId === clip.id; - + const isSelected = selectedClips.some( + (c) => c.trackId === track.id && c.clipId === clip.id + ); return (
{ e.stopPropagation(); - selectClip(track.id, clip.id); + if (e.metaKey || e.ctrlKey || e.shiftKey) { + selectClip(track.id, clip.id, true); + } else if (isSelected) { + deselectClip(track.id, clip.id); + } else { + selectClip(track.id, clip.id, false); + } }} tabIndex={0} onContextMenu={(e) => { diff --git a/apps/web/src/stores/timeline-store.ts b/apps/web/src/stores/timeline-store.ts index ea6fc35..c6ecaea 100644 --- a/apps/web/src/stores/timeline-store.ts +++ b/apps/web/src/stores/timeline-store.ts @@ -21,10 +21,11 @@ export interface TimelineTrack { interface TimelineStore { tracks: TimelineTrack[]; - // Selection - selectedClip: { trackId: string; clipId: string } | null; - selectClip: (trackId: string, clipId: string) => void; - clearSelectedClip: () => void; + // Multi-selection + selectedClips: { trackId: string; clipId: string }[]; + selectClip: (trackId: string, clipId: string, multi?: boolean) => void; + deselectClip: (trackId: string, clipId: string) => void; + clearSelectedClips: () => void; // Actions addTrack: (type: "video" | "audio" | "effects") => string; @@ -55,13 +56,30 @@ interface TimelineStore { export const useTimelineStore = create((set, get) => ({ tracks: [], - selectedClip: null, + selectedClips: [], - selectClip: (trackId, clipId) => { - set({ selectedClip: { trackId, clipId } }); + selectClip: (trackId, clipId, multi = false) => { + set((state) => { + const exists = state.selectedClips.some( + (c) => c.trackId === trackId && c.clipId === clipId + ); + if (multi) { + // Toggle selection + return exists + ? { selectedClips: state.selectedClips.filter((c) => !(c.trackId === trackId && c.clipId === clipId)) } + : { selectedClips: [...state.selectedClips, { trackId, clipId }] }; + } else { + return { selectedClips: [{ trackId, clipId }] }; + } + }); }, - clearSelectedClip: () => { - set({ selectedClip: null }); + deselectClip: (trackId, clipId) => { + set((state) => ({ + selectedClips: state.selectedClips.filter((c) => !(c.trackId === trackId && c.clipId === clipId)), + })); + }, + clearSelectedClips: () => { + set({ selectedClips: [] }); }, addTrack: (type) => {