Closed Bug 1866165 Opened 7 months ago Closed 7 months ago

[wpt-sync] Sync PR 43276 - [GridNG] Honor specified intrinsic sizes in CompleteTrackSizingAlgorithm

Categories

(Core :: Layout: Grid, task, P4)

Product:
Core
Component:
Layout: Grid
Type:
task

Tracking

()

Status:
RESOLVED FIXED
Milestone:
122 Branch
Tracking Flags:
Tracking Status
firefox122 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

(
URL
)

Details

(Whiteboard: [wptsync downstream])

Sync web-platform-tests PR 43276 into mozilla-central (this bug is closed when the sync is complete).

PR: https://github.com/web-platform-tests/wpt/pull/43276
Details from upstream follow.

Ethan Jimenez (from Dev Box) <ethavar@microsoft.com> wrote:

[GridNG] Honor specified intrinsic sizes in CompleteTrackSizingAlgorithm

  1. Previously, whenever we calculated the initial fragment geometry of
    a subgrid, if its intrinsic size was required to determine its
    available space we would default to use an intrinsic fragment
    geometry, making the inline size always indefinite.

    However, it's desirable to compute its actual min/max sizes during
    the execution of CompleteTrackSizingAlgorithm, since tracks might
    change their definitions when their available space changes from
    indefinite to definite, e.g., percentage tracks.

  2. Adding test coverage to the WPT suite for this issue.

Bug: 1475008
Change-Id: I46b207a4eb42781c2147df5710791dc579db3303
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5047584
Commit-Queue: Ethan Jimenez \<ethavar@microsoft.com>
Reviewed-by: Ian Kilpatrick \<ikilpatrick@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1228211}

Component: web-platform-tests → Layout: Grid
Product: Testing → Core
Pushed by wptsync@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/73fc2aac4f2f
[wpt PR 43276] - [GridNG] Honor specified intrinsic sizes in CompleteTrackSizingAlgorithm, a=testonly
Test result changes from PR not available.

https://hg.mozilla.org/mozilla-central/rev/73fc2aac4f2f

Status: NEW → RESOLVED
Closed: 7 months ago
status-firefox122: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 122 Branch
You need to log in before you can comment on or make changes to this bug.