From 630250c23b1cd8ec608c95334ae2babf8e7a00af Mon Sep 17 00:00:00 2001 From: aashishparuvada Date: Mon, 23 Jun 2025 20:51:36 +0530 Subject: [PATCH 1/3] 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) => { From f44f0acf7024febdbbecbbf10772b9b9d3863b4f Mon Sep 17 00:00:00 2001 From: aashishparuvada Date: Mon, 23 Jun 2025 21:16:21 +0530 Subject: [PATCH 2/3] hotfix-improved-selection-standards --- apps/web/src/components/editor/timeline.tsx | 10 +++------- apps/web/src/stores/timeline-store.ts | 3 +++ 2 files changed, 6 insertions(+), 7 deletions(-) diff --git a/apps/web/src/components/editor/timeline.tsx b/apps/web/src/components/editor/timeline.tsx index 1120fbe..e824d42 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, selectedClips, selectClip, deselectClip, clearSelectedClips } = + const { tracks, addTrack, addClipToTrack, removeTrack, toggleTrackMute, removeClipFromTrack, moveClipToTrack, getTotalDuration, selectedClips, selectClip, deselectClip, clearSelectedClips, setSelectedClips } = useTimelineStore(); const { mediaItems, addMediaItem } = useMediaStore(); const { currentTime, duration, seek, setDuration, isPlaying, play, pause, toggle } = usePlaybackStore(); @@ -156,20 +156,18 @@ export function Timeline() { }); 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)); + setSelectedClips(newSelection); } else if (!marquee.additive) { clearSelectedClips(); } setMarquee(null); - }, [marquee, tracks, zoomLevel, selectedClips, selectClip, clearSelectedClips]); + }, [marquee, tracks, zoomLevel, selectedClips, selectClip, clearSelectedClips, setSelectedClips]); const handleDragEnter = (e: React.DragEvent) => { // When something is dragged over the timeline, show overlay @@ -1398,8 +1396,6 @@ function TimelineTrackContent({ e.stopPropagation(); 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); } diff --git a/apps/web/src/stores/timeline-store.ts b/apps/web/src/stores/timeline-store.ts index c6ecaea..4b55a53 100644 --- a/apps/web/src/stores/timeline-store.ts +++ b/apps/web/src/stores/timeline-store.ts @@ -26,6 +26,7 @@ interface TimelineStore { selectClip: (trackId: string, clipId: string, multi?: boolean) => void; deselectClip: (trackId: string, clipId: string) => void; clearSelectedClips: () => void; + setSelectedClips: (clips: { trackId: string; clipId: string }[]) => void; // Actions addTrack: (type: "video" | "audio" | "effects") => string; @@ -82,6 +83,8 @@ export const useTimelineStore = create((set, get) => ({ set({ selectedClips: [] }); }, + setSelectedClips: (clips) => set({ selectedClips: clips }), + addTrack: (type) => { const newTrack: TimelineTrack = { id: crypto.randomUUID(), From e1ffb97be6db8396442fa9f0c42d41b85f597f8c Mon Sep 17 00:00:00 2001 From: aashishparuvada Date: Mon, 23 Jun 2025 21:24:51 +0530 Subject: [PATCH 3/3] hotfix:removed-redundant-div --- apps/web/src/components/editor/timeline.tsx | 51 ++++++++------------- 1 file changed, 18 insertions(+), 33 deletions(-) diff --git a/apps/web/src/components/editor/timeline.tsx b/apps/web/src/components/editor/timeline.tsx index e548a50..675b398 100644 --- a/apps/web/src/components/editor/timeline.tsx +++ b/apps/web/src/components/editor/timeline.tsx @@ -132,7 +132,6 @@ export function Timeline() { // 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(); @@ -140,7 +139,17 @@ export function Timeline() { 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 + // Validation: skip if too small + if (Math.abs(x2 - x1) < 5 || Math.abs(y2 - y1) < 5) { + setMarquee(null); + return; + } + // Clamp to timeline bounds + const clamp = (val: number, min: number, max: number) => Math.max(min, Math.min(max, val)); + const bx1 = clamp(x1, 0, rect.width); + const bx2 = clamp(x2, 0, rect.width); + const by1 = clamp(y1, 0, rect.height); + const by2 = clamp(y2, 0, rect.height); let newSelection: { trackId: string; clipId: string }[] = []; tracks.forEach((track, trackIdx) => { track.clips.forEach((clip) => { @@ -150,12 +159,11 @@ export function Timeline() { const clipTop = trackIdx * 60; const clipBottom = clipTop + 60; const clipRight = clipLeft + clipWidth; - // Check intersection if ( - x1 < clipRight && - x2 > clipLeft && - y1 < clipBottom && - y2 > clipTop + bx1 < clipRight && + bx2 > clipLeft && + by1 < clipBottom && + by2 > clipTop ) { newSelection.push({ trackId: track.id, clipId: clip.id }); } @@ -163,10 +171,10 @@ export function Timeline() { }); if (newSelection.length > 0) { if (marquee.additive) { - const current = new Set(selectedClips.map((c) => c.trackId + ":" + c.clipId)); + const selectedSet = new Set(selectedClips.map((c) => c.trackId + ':' + c.clipId)); newSelection = [ ...selectedClips, - ...newSelection.filter((c) => !current.has(c.trackId + ":" + c.clipId)), + ...newSelection.filter((c) => !selectedSet.has(c.trackId + ':' + c.clipId)), ]; } setSelectedClips(newSelection); @@ -174,7 +182,7 @@ export function Timeline() { clearSelectedClips(); } setMarquee(null); - }, [marquee, tracks, zoomLevel, selectedClips, selectClip, clearSelectedClips, setSelectedClips]); + }, [marquee, tracks, zoomLevel, selectedClips, setSelectedClips, clearSelectedClips]); const handleDragEnter = (e: React.DragEvent) => { // When something is dragged over the timeline, show overlay @@ -619,29 +627,6 @@ export function Timeline() { onMouseDown={handleTimelineMouseDown} onWheel={handleWheel} > - {/* Overlay for deselect/marquee */} -
- {/* Marquee selection rectangle */} - {marquee && marquee.active && timelineRef.current && ( -
- )} {tracks.length === 0 ? (