Closed Bug 1176808 Opened 9 years ago Closed 7 years ago

Computed style for flex item in inspector is still min-height:0px.

Categories

(Core :: CSS Parsing and Computation, defect)

defect
Not set
normal

Tracking

()

RESOLVED DUPLICATE of bug 1304636

People

(Reporter: alice0775, Unassigned)

References

Details

Steps to reproduce:
1. Open page that using Flex box
    Bug 1162401 attachment 8602816 [details] ( .container is using flex box)
    or
    Bug 1176723 http://codepen.io/anon/pen/zGEgQE (the 2cd div is using flex box)
2. Inspect the flex box's computed style

Actual Results:
min-height:0px

Expected Results:
Computed style for flex box in inspector should be min-height:suto.
This is from:
http://mxr.mozilla.org/mozilla-central/source/layout/style/nsComputedDOMStyle.cpp?rev=d80bdc49a0d5&mark=4343-4348#4343

Per the XXX comment there (and the similar code in DoGetMinWidth()), that code currently thinks "min-[height|width]:auto" should compute to "-moz-min-content" on a flex item, because that's what the spec used to say.

Nowadays, the spec says it resolves to something more complex, which we compute during reflow. Its computed value should just be 0 for non-flex/grid-items and "auto" for flex/grid items.
Status: UNCONFIRMED → NEW
Component: Developer Tools: Inspector → CSS Parsing and Computation
Ever confirmed: true
Product: Firefox → Core
Summary: Computed style for flex box in inspector is still min-height:0px. → Computed style for flex item in inspector is still min-height:0px.
Assignee: nobody → dholbert
This was independently reported (and then fixed) in bug 1304636. --> Duping to that bug where this was fixed.
Assignee: dholbert → nobody
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.