Skip to content

WebKit export of https://bugs.webkit.org/show_bug.cgi?id=322298 - #64169

Open
annevk wants to merge 1 commit into
web-platform-tests:masterfrom
annevk:wpt-export-for-webkit-322298
Open

annevk wants to merge 1 commit into
web-platform-tests:masterfrom
annevk:wpt-export-for-webkit-322298

Conversation

@annevk

@annevk annevk commented Oct 25, 2026

Copy link
Copy Markdown
Member

WebKit export from bug: [block] align-content moves a float child twice

[block] align-content moves a float child twice

align-content distributes a block container's extra space by moving its content. shiftForAlignContent moves
every child box and FloatingObjects::shiftFloatsBy then moves the float renderers again, so a float that is a
child of the container travels twice as far as the content it sits with.

A float taken from a child's float list moves twice as well: the child moves with the rest of the content and
shiftFloatsBy then moves the float's renderer, which belongs to that child. Only the FloatingObject's frame
rect needs the shift there, since that rect is in this container's coordinate space.

WebKit-Bug: https://bugs.webkit.org/show_bug.cgi?id=322298
WebKit-Canonical-Link: https://commits.webkit.org/319665@main

align-content distributes a block container's extra space by moving its content. shiftForAlignContent moves
every child box and FloatingObjects::shiftFloatsBy then moves the float renderers again, so a float that is a
child of the container travels twice as far as the content it sits with.

A float taken from a child's float list moves twice as well: the child moves with the rest of the content and
shiftFloatsBy then moves the float's renderer, which belongs to that child. Only the FloatingObject's frame
rect needs the shift there, since that rect is in this container's coordinate space.

WebKit-Bug: https://bugs.webkit.org/show_bug.cgi?id=322298
WebKit-Canonical-Link: https://commits.webkit.org/319665@main

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

Copy link
Copy Markdown
Collaborator

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 WebKit project.

@annevk
annevk enabled auto-merge (rebase) October 25, 2026 08:54

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants