Repository navigation
fix(native): HTML renders on iOS, tweet photos that fill their box - #477
Merged
Merged
Maple Review Bot / Maple / review
succeeded
Nov 12, 2026 in 59s
Confidence 10/10 · No issues found
🟢 Confidence 10/10 · safe to merge. The delta is four lines moving the portal clip reset ahead of the iframe branch, pinned by a test that puts the fixed frame in a scrolling list.
no findings · tests covered · risk medium
Moves the portal clip reset ahead of the iframe branch in walk, so a fixed HTML render inside a scrolling list is no longer clipped by it, and extends the layer test to cover that case. Safe to merge.
What was checked
own_clipsnow covers the iframe branch (frames.rs:60), so a fixed iframe carries no ancestor clips;a_fixed_frame_is_its_own_layerassertsclipsis empty.scrolls_xneeds a non-wrapping flex row (layout.rs:708), so the new clip inwalknever applies to a code block or table.is_overcompares the sticky's layer to the frame's (frames.rs:101), keeping a page's pinned date off a dialog's frame.
4fccfb6 · Reply "won't fix" to dismiss a finding, or mention @maple-review-bot to ask about one.
Loading