Closed
Bug 1191155
Opened 9 years ago
Closed 5 years ago
flex-basis percent values should be resolved (instead of treated as 'auto') in quirks mode
Categories
(Core :: Layout, defect)
Core
Layout
Tracking
()
VERIFIED
INVALID
Tracking | Status | |
---|---|---|
firefox42 | --- | affected |
People
(Reporter: dholbert, Unassigned)
Details
Attachments
(2 files, 1 obsolete file)
STR: 1. Load attached testcase EXPECTED RESULTS: No red visible. ACTUAL RESULTS: Red is visible. The testcase is in quirks mode (because there's no !DOCTYPE), so vertical percent values should be resolved (not treated as 'auto') in an auto-height container. See upcoming reference case w/ block layout instead of flex layout. (This bug is the "separately, it looks like Firefox does not..." parenthetical mentioned at the end of this www-style post from biesi: https://lists.w3.org/Archives/Public/www-style/2015Aug/0027.html )
Reporter | ||
Comment 1•9 years ago
|
||
[er, tweaking prose in testcase to just say "element" instead of "flex item", so the reference case (with no flex stuff) can use the same language without confusion]
Attachment #8643409 -
Attachment is obsolete: true
Reporter | ||
Comment 2•9 years ago
|
||
Comment 3•9 years ago
|
||
> vertical percent values should be resolved (not treated as 'auto') in an auto-height > container Should they? The quirk at https://quirks.spec.whatwg.org/#the-percentage-height-calculation-quirk is restricted to percentage values of the 'height' property only, and I don't think we should be randomly extending it to other situations.
Reporter | ||
Comment 4•5 years ago
|
||
Yeah, let's leave this as-is. Our current behavior (on the testcase) is interoperable with Blink, and there's no compelling reason to add/extend quirks in a direction away from compatibility.
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → INVALID
Yeah, I think we changed our (blink's) behavior since that mailing list post in comment 0, and I've come around to the view that quirks mode should not affect this.
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•