Skip to content

Conversation

chromium-wpt-export-bot
Copy link
Collaborator

This CL implements out-of-flow support in masonry (outside of
fragmentation).

  • Enhanced placeOutOfFlowItems to properly handle OOF items with
    correct containing block calculation, alignment offset computation,
    and static positioning.
  • Extended out_of_flow_layout_part to recognize masonry containers
    and delegate OOF item placement to the masonry-specific containing
    block calculation.

Bug: 343257585
Change-Id: I0ecccb7b2e6c3a46a810bd01577a88d53b05d7a1
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6909974
Reviewed-by: Kurt Catti-Schmidt <[email protected]>
Commit-Queue: Yanling Wang <[email protected]>
Reviewed-by: Alison Maher <[email protected]>
Cr-Commit-Position: refs/heads/main@{#1511947}

This CL implements out-of-flow support in masonry (outside of
fragmentation).
- Enhanced `placeOutOfFlowItems` to properly handle OOF items with
correct containing block calculation, alignment offset computation,
and static positioning.
- Extended `out_of_flow_layout_part` to recognize masonry containers
and delegate OOF item placement to the masonry-specific containing
block calculation.

Bug: 343257585
Change-Id: I0ecccb7b2e6c3a46a810bd01577a88d53b05d7a1
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6909974
Reviewed-by: Kurt Catti-Schmidt <[email protected]>
Commit-Queue: Yanling Wang <[email protected]>
Reviewed-by: Alison Maher <[email protected]>
Cr-Commit-Position: refs/heads/main@{#1511947}
Copy link
Collaborator

@wpt-pr-bot wpt-pr-bot left a comment

Choose a reason for hiding this comment

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

The review process for this patch is being conducted in the Chromium project.

@chromium-wpt-export-bot chromium-wpt-export-bot merged commit f554aed into master Sep 6, 2025
24 checks passed
@chromium-wpt-export-bot chromium-wpt-export-bot deleted the chromium-export-f4fac70f9e branch September 6, 2025 00:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants