summaryrefslogtreecommitdiff
path: root/src/audio/idct.cpp
diff options
context:
space:
mode:
authorskal <pascal.massimino@gmail.com>2026-02-05 21:48:38 +0100
committerskal <pascal.massimino@gmail.com>2026-02-05 21:48:38 +0100
commit58ad5c16d6f1b2afc51ec5b36ed08643643e5fd0 (patch)
tree14526c89efcae26a87803c4d5d4c06df9ac95ea1 /src/audio/idct.cpp
parentbecb354b49119d27082b542ed7bdc70bc606e3b3 (diff)
fix(timeline-editor): Remove time indicator and fix sequence overlap
Bug Fix #1: Remove Current Time Indicator - Removed vertical time indicator line (user feedback: didn't look good) - Deleted .current-time-indicator CSS class - Removed HTML element from timeline-container - Cleaner visual appearance without the distracting line Bug Fix #2: Dynamic Y Positioning to Prevent Overlap - Sequences now use cumulative Y positioning instead of fixed spacing - Previously: seqIndex * 80px (fixed spacing caused overlaps) - Now: Accumulate Y position based on actual sequence heights Algorithm: 1. Track cumulativeY starting at 0 2. Calculate each sequence height dynamically: height = max(70, 20 + numEffects * 30 + 5) 3. Position sequence at cumulativeY 4. Store Y position in seq._yPosition 5. Increment: cumulativeY += height + 10px gap 6. Effects use seq._yPosition + offset 7. Scroll handler uses seq._yPosition for target Example (3 sequences with different effect counts): Before (overlap): After (no overlap): Seq0: y=0, h=70 Seq0: y=0, h=70 Seq1: y=80, h=120 ✗ Seq1: y=80, h=120 Seq2: y=160 (hidden) Seq2: y=210, h=70 ✓ Technical Changes: - Added cumulativeY tracking variable - Added sequenceGap = 10px constant - Sequences store _yPosition property - Effects use seq._yPosition instead of seqIndex * 80 - Wheel scroll uses sequences[i]._yPosition - Properly handles varying sequence heights Result: Sequences never overlap, regardless of effect count
Diffstat (limited to 'src/audio/idct.cpp')
0 files changed, 0 insertions, 0 deletions