Closed Bug 1434380 Opened 2 years ago Closed 2 years ago

Adjust sizes in grid-auto-min-sizing-definite-001-ref.html to be more grounded & easier to adjust

Categories

(Core :: Layout, enhancement)

enhancement
Not set

Tracking

()

RESOLVED FIXED
mozilla60
Tracking Status
firefox60 --- fixed

People

(Reporter: dholbert, Assigned: dholbert)

References

Details

Attachments

(1 file)

In bug 958714, I'm going to need to adjust the values in this reference case:
https://dxr.mozilla.org/mozilla-central/source/layout/reftests/css-grid/grid-auto-min-sizing-definite-001-ref.html

...and I had a very hard time figuring out how to adjust them correctly, in part because the current sizes are a bit arbitrary & don't all correspond directly to pieces of the testcase, AFAICT.

I'm filing this bug to just to a bit of cleanup in this file (without affecting its rendering), so that the changes in 958714 can be a bit more surgical & understandable.
Comment on attachment 8946763 [details]
Bug 1434380: Rewrite parts of reftest reference case grid-auto-min-sizing-definite-001-ref.html to be easier to understand & adjust.

https://reviewboard.mozilla.org/r/216702/#review222796
Attachment #8946763 - Flags: review?(mats) → review+
Pushed by dholbert@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/57df747377cc
Rewrite parts of reftest reference case grid-auto-min-sizing-definite-001-ref.html to be easier to understand & adjust. r=mats
https://hg.mozilla.org/mozilla-central/rev/57df747377cc
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla60
You need to log in before you can comment on or make changes to this bug.