[webkit-changes] [WebKit/WebKit] f2e73e: Grid track sizing should reset the override width ...
mattwoodrow
noreply at github.com
Sun Nov 13 16:01:16 PST 2022
Branch: refs/heads/main
Home: https://github.com/WebKit/WebKit
Commit: f2e73e71c88c14822ac5f6d077634fc828ad686e
https://github.com/WebKit/WebKit/commit/f2e73e71c88c14822ac5f6d077634fc828ad686e
Author: Matt Woodrow <mattwoodrow at apple.com>
Date: 2022-11-13 (Sun, 13 Nov 2022)
Changed paths:
A LayoutTests/fast/dynamic/relayout-grid-should-reset-instrinsic-sizes-expected.html
A LayoutTests/fast/dynamic/relayout-grid-should-reset-instrinsic-sizes.html
M Source/WebCore/rendering/GridTrackSizingAlgorithm.cpp
M Source/WebCore/rendering/RenderGrid.cpp
Log Message:
-----------
Grid track sizing should reset the override width as well as height to compute intrinsic sizing.
https://bugs.webkit.org/show_bug.cgi?id=247465
<rdar://101304306>
Reviewed by Alan Baradlay.
Grid layout (sometimes) sets the override content size and containing block sizes, so that grid items get sized relative to their grid tracks, rather than the size of the grid element.
During the sizing process, we want to know the intrinsic size of children, so we reset any overrides and re-layout children to measure them.
We're currently not resetting the content logical width, and there's a comment saying that grid never uses it, but that's out of date and we do need to reset this.
* LayoutTests/fast/dynamic/relayout-grid-should-reset-instrinsic-sizes-expected.html: Added.
* LayoutTests/fast/dynamic/relayout-grid-should-reset-instrinsic-sizes.html: Added.
* Source/WebCore/rendering/GridTrackSizingAlgorithm.cpp:
(WebCore::GridTrackSizingAlgorithmStrategy::logicalHeightForChild const):
* Source/WebCore/rendering/RenderGrid.cpp:
Canonical link: https://commits.webkit.org/256622@main
More information about the webkit-changes
mailing list