1690422 - Incorrect calculation of the inline min/max content contribution in grid tracking algorithm when using aspect-ratio property in grid item

Status: RESOLVED FIXED

(Core :: Layout: Grid, defect, P2)

Description

[Boris Chiou [:boris]]

When using aspect-ratio on grid items, we calculate the size of this grid correctly, but its grid tracking size doesn't take this aspect-ratio property into account.
e.g.

.grid-container {
  display: grid;
  grid-template-columns: auto;
  width: max-content;
  height: 100px;
  gap: 1rem;
}
.grid-item {
  aspect-ratio: 1/1;
  height: 100px;
  width: auto;  // the final width should be 100px (= 100px * 1 / 1).
  min-width: 0px;
}

<div class="grid-container">
  <div class="grid-item">
    <div style="width:50px; height:50px;"></div>
  </div>
</div>

The max content contribution of this grid item should be 100px. However, the actual result is 50px because IntrinsicForAxis() and AddIntrinsicSizeOffset() don't use aspect-ratio to replace the min-content and max-content values. This is a follow-up bug of Bug 1646100 for Grid layout.

Besides, the usage of AspectRatio in IntrinsicForAxis() should take box-sizing into account. We always use content-box for now, in all cases. This is not correct. This is a follow-up bug for Bug 1683424.

Comment 1

[Boris Chiou [:boris]]

Attachment: Bug 1690422 - Take box-sizing into account when computing the intrinsic size with the preferred aspect ratio.

Comment 2

[Boris Chiou [:boris]]

Attachment: Bug 1690422 - Resolve {min|max}-content contribution with the preferred aspect ratio and the definite block size.

This is simliar with the previous patch, but we use
min-content/max-content for the inner div (i.e. child frame).

Perhaps we could remove the simliar implementation from
ComputeISizeValue(). However, I still keep it because it can avoid some
extra work when resolving {min|max}-content with aspect-ratio when
reflowing.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/373fe3a0806a
Take box-sizing into account when computing the intrinsic size with the preferred aspect ratio. r=TYLin,mats
https://hg.mozilla.org/integration/autoland/rev/d5975841c326
Resolve {min|max}-content contribution with the preferred aspect ratio and the definite block size. r=TYLin

Comment 4

[Web Platform Test Sync Bot (Matrix: #interop:mozilla.org)]

Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/27655 for changes under testing/web-platform/tests

Comment 5

[Noemi Erli[:noemi_erli]]

https://hg.mozilla.org/mozilla-central/rev/373fe3a0806a
https://hg.mozilla.org/mozilla-central/rev/d5975841c326

Comment 6

[Web Platform Test Sync Bot (Matrix: #interop:mozilla.org)]

Upstream PR merged by moz-wptsync-bot