Closed
Bug 1191155
Opened 10 years ago
Closed 6 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•10 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•10 years ago
|
||
Comment 3•10 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•6 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: 6 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
•