feat: add interactive dual-thumb timeline range slider with live video scrubbing#865
feat: add interactive dual-thumb timeline range slider with live video scrubbing#865SatyaViswas wants to merge 3 commits into
Conversation
|
@SatyaViswas is attempting to deploy a commit to the magic-peach1's projects Team on Vercel. A member of the Team first needs to authorize it. |
👋 Thanks for your PR, @SatyaViswas!Welcome to Reframe — a browser-based video editor built for everyone 🎬
What happens next
Quick checklist
Useful links
Happy coding! 🎉 |
✅ PR Format Check Passed — @SatyaViswasBasic format checks passed. A maintainer will review your code changes. This does not mean the PR is approved — it just means the format is correct. |
|
Hey @SatyaViswas! CI hasn't run on this PR yet. Please rebase on git fetch origin
git rebase origin/main
git push --force-with-leaseAlso, please attach a screen recording showing your change working in the browser (see CONTRIBUTING.md for instructions). Both CI and a recording are required before a PR can be merged. |
9aa66bc to
086b11f
Compare
|
Hey @magic-peach , I have resolved the conflicts on this branch! Also I attached the screen recording before already, but I am attaching again. Screen.Recording.2026-05-21.at.7.28.16.PM.mov |
086b11f to
95ddcc1
Compare
|
Hey @magic-peach , just to remind, I resolved all conflicts and my PR is upto date with the main branch. Please review and merge my PR! |
95ddcc1 to
b2b4fc9
Compare
|
Hey @magic-peach. I resolved all the latest conflicts and my PR is upto date with the main branch. Please review and merge my PR! |
b2b4fc9 to
3225a06
Compare
|
Hey @magic-peach , could you please review this PR and close it, so that I could work on new issue! I also solved the Application error shown in image which occurs when we use adjustment or trim options too many times: |
f43154c to
fb90293
Compare
|
@SatyaViswas This PR has merge conflicts with Please rebase your branch on git fetch origin
git rebase origin/main
git push --force-with-lease origin feat/timeline-trim-sliderAfter rebasing, CI will run automatically. The interactive dual-thumb slider feature looks promising — once conflicts are resolved and CI passes, this will be reviewed for merge. |
fb90293 to
4d8337f
Compare
|
Hey @magic-peach , I have solved the conflicts. Please review and merge the PR! |
cfa03f7 to
fc1492c
Compare
|
Hey @magic-peach , I have resolved all conflicts in my PR. Please review and merge it! |
|
Hey @magic-peach , I have also fixed the ffmpeg export error occurring in the main branch. Please review my PR! |
49e2abf to
947463c
Compare
|
Hey @magic-peach , you recently merged my PR #1199 , which fixed the ffmpeg export error. But I guess another PR merge by someone else made the ffmpeg export error return. But in this PR, the exporting still works without any error. I have also solved all CI build issues too.Please review and merge my PR! |
|
Also this is the new UI change in trim section: Screen.Recording.2026-05-27.at.10.20.06.AM.mov |
da62186 to
2cc81d1
Compare
2cc81d1 to
9f1a900
Compare

Description
This PR introduces an interactive, dual-thumb range slider overlay directly on top of the trimming timeline to drastically elevate the visual editing workflow.
Key changes include:
Related Issue
Fixes #537
Type of Contribution
Participant Info
Screen Recording
Before:
Screen.Recording.2026-05-21.at.7.30.34.PM.mov
After:
Screen.Recording.2026-05-21.at.7.28.16.PM.mov
Checklist
bun run lintpasses (no ESLint errors)bunx tsc --noEmitpasses (no TypeScript errors)aria-label/ accessible namesconsole.logstatements left in