Skip to content

@remotion/studio: Remove splitter hover state - #9956

Merged
JonnyBurger merged 1 commit into
mainfrom
codex/tone-down-studio-splitter-hover
Jul 30, 2026
Merged

@remotion/studio: Remove splitter hover state#9956
JonnyBurger merged 1 commit into
mainfrom
codex/tone-down-studio-splitter-hover

Conversation

@JonnyBurger

Copy link
Copy Markdown
Member

Summary

  • keep the existing thin black Studio splitter dividers
  • remove the blue hover and active splitter styles
  • remove the redundant hover state listeners and class toggles

Testing

  • bun run build
  • bun run stylecheck

@vercel

vercel Bot commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
bugs Ready Ready Preview Jul 30, 2026 5:23pm
remotion Ready Ready Preview Jul 30, 2026 5:23pm

Request Review

@pullfrog pullfrog Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

✅ No new issues found.

Reviewed changes — cosmetic simplification of Studio splitter handles: removed blue hover/active CSS styles, removed the hover-state tracking useEffect, and consolidated two-tier (unhovered vs active/hovered) CSS into single rules with the existing black gradient and 2x scale factor.

  • SplitterHandle.tsx — removed classList.add/remove('remotion-splitter-active') calls from drag start/end, and removed the entire useEffect that managed mouseenter/mouseleave/mousedown/mouseup listeners for remotion-splitter-hover class toggling.
  • inject-css.ts — merged unhoveredDragAreaFactor and hoveredDragAreaFactor into a single dragAreaFactor (2), removed the .remotion-splitter-active/.remotion-splitter-hover blue-gradient CSS rules on both horizontal and vertical variants, and dropped the z-index: 1000.

Pullfrog  | View workflow run | Using DeepSeek Pro (free via Pullfrog for OSS) (Claude Opus not used — the program covers this model; add its provider key to run your pick) | 𝕏

@JonnyBurger
JonnyBurger merged commit 3bee2cf into main Jul 30, 2026
19 checks passed
@JonnyBurger
JonnyBurger deleted the codex/tone-down-studio-splitter-hover branch July 30, 2026 17:24
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.

1 participant