Skip to content

#6870 Studio: <Img> should also show up as a layer#6892

Merged
JonnyBurger merged 3 commits intoremotion-dev:mainfrom
tiwariaayu:#6870
Mar 25, 2026
Merged

#6870 Studio: <Img> should also show up as a layer#6892
JonnyBurger merged 3 commits intoremotion-dev:mainfrom
tiwariaayu:#6870

Conversation

@tiwariaayu
Copy link
Copy Markdown
Contributor

@tiwariaayu tiwariaayu commented Mar 20, 2026

How it Works
When is rendered:

  • useImageInTimeline registers it with SequenceManager.
  • Studio detects type 'image' → applies blue gradient styling.
  • stack prop enables “Open in Editor.”
  • Public folder images link directly to the asset in the sidebar.

Verification

  • Manual Testing: components appear in the timeline with correct labels and styling in Remotion Studio.
  • Unit Tests: All 307 core tests passed, including new tests for registration.
  • Build: Project built successfully using bunx turbo run make.

Closes #6870

@pullfrog
Copy link
Copy Markdown
Contributor

pullfrog bot commented Mar 20, 2026

Review blocked — PR branch not accessible

I was unable to review PR #6870 (Studio: <Img> should also show up as a layer) because the branch pull/6870/head is not available in this environment. This typically means the PR is from a fork and the branch hasn't been pushed to a remote this workflow can access.

To unblock review, one of the following is needed:

  1. Push the PR branch to the upstream remote so it can be fetched
  2. Provide a direct link to the diff or a rendered version of the changes
  3. Share the specific files changed so I can review them directly

Once the branch is accessible, I can complete the review.

Pullfrog  | View workflow run | Triggered by Pullfrogpullfrog.com𝕏

@vercel
Copy link
Copy Markdown
Contributor

vercel bot commented Mar 20, 2026

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

Project Deployment Actions Updated (UTC)
bugs Ready Ready Preview, Comment Mar 25, 2026 11:58am
remotion Ready Ready Preview, Comment Mar 25, 2026 11:58am

Request Review

@JonnyBurger
Copy link
Copy Markdown
Member

Thanks, will look at it after #6826 is merged

- Add TimelineImageInfo component that renders image thumbnails using canvas repeating pattern
- Give image sequences the same 50px height as video in the timeline
- Refactor getTimelineLayerHeight to accept sequence type directly, removing repeated ternaries
- Add simple-img test composition
- Exclude image sequences from frame number overlay

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@JonnyBurger JonnyBurger enabled auto-merge (squash) March 25, 2026 13:10
@JonnyBurger
Copy link
Copy Markdown
Member

Thanks for the contribution @tiwariaayu! I've added image thumbnail rendering in the timeline using a canvas repeating pattern (so the image tiles across the bar, similar to how video frames work), refactored the height calculation to use a single source of truth, and added a test composition. Enabled auto-merge 🎉

@JonnyBurger JonnyBurger merged commit d08c4f5 into remotion-dev:main Mar 25, 2026
17 of 18 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.

Studio: <Img> should also show up as a layer

2 participants