Skip to content

fix: correct Skia mirrored frame rotation order - #4220

Open
mfkrause wants to merge 1 commit into
margelo:mainfrom
mfkrause:fix/skia-mirrored-frame-rotation
Open

mfkrause wants to merge 1 commit into
margelo:mainfrom
mfkrause:fix/skia-mirrored-frame-rotation

Conversation

@mfkrause

Copy link
Copy Markdown

SkiaCamera can show the front camera preview upside down when frame.isMirrored is true and frame.orientation is left or right.

To reproduce, use SkiaCamera with the front camera on an iPhone. Keep enablePhysicalBufferRotation at its default value and immediately draw each camera frame to the canvas without any modifications. The issue occurs when a frame is both mirrored and rotated by 90掳 or 270掳.

The renderer currently applies mirroring and rotation in the wrong order. This fix corrects that order.

@vercel

vercel Bot commented Oct 14, 2026

Copy link
Copy Markdown

@mfkrause is attempting to deploy a commit to the Margelo Team on Vercel.

A member of the Team first needs to authorize it.

This branch has not been deployed

No deployments
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