Inline-size is too wide for empty <select> lists compared to other UAs

RESOLVED FIXED in Firefox 64

Status

()

enhancement
P4
minor
RESOLVED FIXED
10 months ago
10 months ago

People

(Reporter: mats, Assigned: mats)

Tracking

({testcase})

Trunk
mozilla64
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox64 fixed)

Details

Attachments

(2 attachments, 1 obsolete attachment)

Posted file Testcase (obsolete) —
Edge and Chrome renders the testcase as expected: the <select>
content-box width appears to be zero in all cases.

I tend to think we should follow their example.
(Safari renders the first row a bit like us, but fails
completely in the 2nd row)

Note, you need the fix in bug 1499127 for the height
to be correct in the first row.
Posted file Testcase
Attachment #9017351 - Attachment is obsolete: true
This patch also makes the reference for the test added in bug 1499127
render correctly, so we can enable that test now.
Attachment #9018033 - Flags: review?(emilio)
Comment on attachment 9018033 [details] [diff] [review]
Make empty <select>s have zero content-box inline-size, like in other UAs

Review of attachment 9018033 [details] [diff] [review]:
-----------------------------------------------------------------

So, this looks like it was introduced initially in:

  https://github.com/mozilla/gecko-dev/commit/e5db62821d3cc51cc603683095279e697f1982b4

which has no bug number or anything else

Then changed to two X in:

  https://github.com/mozilla/gecko-dev/commit/702462fbfe83be16edb7737a4c1f68bcebca5672

Which is bug 28593...

Then Boris actually changed it for the :empty rule we have today.

Anyway, this looks fine to me, thanks!
Attachment #9018033 - Flags: review?(emilio) → review+
Pushed by mpalmgren@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/a9b1d69e379a
Make empty <select>s have zero content-box inline-size, like in other UAs.  r=emilio
https://hg.mozilla.org/mozilla-central/rev/a9b1d69e379a
Status: NEW → RESOLVED
Closed: 10 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla64
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/13758 for changes under testing/web-platform/tests
Upstream PR merged
You need to log in before you can comment on or make changes to this bug.