Skip to content

feat(editor): add visual adjustments and fades - #3

Merged
sjungwon03 merged 1 commit into
mainfrom
feat/464-visual-adjustments-fades
Sep 27, 2026
Merged

sjungwon03 merged 1 commit into
mainfrom
feat/464-visual-adjustments-fades

Conversation

@sjungwon03

Copy link
Copy Markdown
Member

Summary

  • Add per-visual-clip brightness, contrast, saturation, opacity, and fade-in/out controls in the web editor inspector.
  • Composite overlapping visual layers in insertion order for preview and local WebM export; the same pure model computes opacity, filter, and effective audio gain.
  • Preserve outer fades across clip splits and reduce fades when trimmed, and keep the inspector available at narrower widths.
  • Update the fork README to describe the supported basic adjustments and remaining Classic catalog limits.

Verification

  • apps/web TypeScript check passed.
  • apps/web Vitest 13/13 passed, including overlap/trim rules and mocked layered browser export.
  • apps/web Vite production build passed.
  • No browser executable is attached for a manual rendered-media check; CI and reviewer should inspect preview/export parity.

Integration

This PR targets Theorvane/OpenCut main only. After review and merge, OpenScene issue Theorvane/OpenScene#464 will advance the pinned submodule in a separate PR. No upstream OpenCut PR is planned.

@sjungwon03-ai sjungwon03-ai left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review summary

Verdict: Approved

  • Reviewed: visual-layer compositing, adjustment and fade model, browser preview, WebM export, and trim/split behavior.
  • Checks: All four GitHub Actions checks passed; local npm test (13/13), npm run build, and tsc --noEmit passed.

@sjungwon03
sjungwon03 merged commit 4eb332f into main Sep 27, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants