Closed Bug 1574060 Opened 2 months ago Closed 2 months ago

Implement layout for 'display: flow-root list-item'

Categories

(Core :: Layout: Generated Content, Lists, and Counters, enhancement, P2)

70 Branch
enhancement

Tracking

()

RESOLVED FIXED
mozilla70
Tracking Status
firefox70 --- fixed

People

(Reporter: mats, Assigned: mats)

References

Details

(Keywords: dev-doc-complete)

Attachments

(1 file)

I missed that 'display: flow-root list-item' is actually a distinct value with different layout than 'display: list-item' (in the same way flow-root is different from block).

Should be a trivial fix...

Pushed by mpalmgren@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/e3f6e2f3e216
Implement layout for 'display: flow-root list-item'.  r=emilio
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/18488 for changes under testing/web-platform/tests
Upstream web-platform-tests status checks passed, PR will merge once commit reaches central.

Rachel, FYI, display: flow-root list-item is now implemented correctly. (I kind of forgot about it in bug 1105868, oops.)
(It behaves the same as list-item except for the differences that we normally have between blockvs flow-root.)

I found a MDN document about BFCs here: https://developer.mozilla.org/en-US/docs/Web/CSS/CSS_Flow_Layout/Intro_to_formatting_contexts so I mentioned the new value there. (I'll leave the documentation of displayin general to you, since it's probably easier if you edit in all the new syntax changes / values at the same time.)

Keywords: dev-doc-needed
Type: defect → enhancement
Status: NEW → RESOLVED
Closed: 2 months ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.