[bartlomiejbloniarz]

Problem

On iOS, an inline video preview is sometimes drawn zoomed in and cropped instead of letterboxed. It stays that way until the view is recreated. This happens on main, for example in PR descriptions with portrait recordings.

expo-image draws a frame object as soon as it is set, with the contentFit the view has at that moment. A preview frame that arrives in the same update that mounts the image view is drawn with the default cover fit, even though the view asks for contain.

Change

VideoThumbnailImage keeps its image view mounted and passes the frame only after the view's first layout. By then contentFit is applied, so the frame is drawn with the requested fit.

Related PRs

This follows #25988, which kept before/after video tables intact in PR descriptions. #32196 fixes a separate bug where previews never load on mobile; the two PRs are independent. A fix for videos keeping their own aspect ratio will follow once these merge.

Scope and approval

A small fix of a visible bug: portrait previews are shown zoomed in. It changes only VideoThumbnailImage.

Verification

• vp test run src/lib/videoThumbnails.test.ts src/lib/filePreview.test.ts src/components in apps/mobile: passes.
• vp run typecheck in apps/mobile, vp lint and vp fmt --check on the changed file: pass.
• iOS simulator, on current main and with this change: the same portrait recording's preview is cropped before and letterboxed after.

iOS — preview of a portrait recording in a PR description

Before	After

Claude Opus 5.5 in Claude Code, running in T3 Code.

[macroscopeapp]

Approvability

Verdict: Approved at b22bb98

Macroscope's review found this PR approvable — Video thumbnails now receive their frame after layout so iOS preserves the intended fit. The change is limited to preview rendering and leaves loading behavior and product defaults unchanged.

^{You can add or adjust custom eligibility rules. Learn more.}

[coderabbitai]

No actionable comments were generated in the recent review.

Walkthrough

VideoThumbnailImage now renders its image view with a null source until layout completes. After layout, it sets the source to the loaded or cached thumbnail.

Changes: Thumbnail image layout

Layer / File(s)	Summary
Keep the image mounted until layout apps/mobile/src/components/VideoThumbnailImage.tsx	The component tracks whether the image view has laid out. It renders the image with a null source before layout, then uses the thumbnail source. Previously, it returned null when no thumbnail was available.

Estimated code review effort: 2 (Simple) | ~8 minutes

Change: Bug fix

Suggested labels: bugfix, size:S

Suggested reviewers: juliusmarminge

Merge Risk: Minimal · up to b22bb

Video previews defer their frame until layout. No merge-blocking regression was established.

Pre-merge checks: 4

Check name	Status	Explanation
Title check		The title clearly describes the iOS video preview fit fix.
Linked Issues check		Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check		Check skipped because no linked issues were found for this pull request.
Description check		The description covers the problem, change, scope, and verification. It includes before and after screenshots and identifies the agent used.

Finishing Touches

Generate unit tests (beta)

• Create a new PR

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Recent review info

Run configuration

• Source mode: Auto
• Repository: pingdotgg/t3code
• Branch: main
• File: .coderabbit.config.ts
• Commit: b90e554
• Layers: Repository TS config
• Review profile: CHILL
• Plan: Advanced
• Run ID: f540b648-72eb-4c7b-9329-c7f403a0939e

Commits

Reviewing files that changed from the base of the PR and between b90e554 and b22bb98.

Files selected for processing: 1

• apps/mobile/src/components/VideoThumbnailImage.tsx

Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 9 remain after this review.