unprefix min-content, max-content, fit-content, and fill keywords

NEW
Unassigned

Status

()

Core
Layout
P3
normal
a year ago
3 months ago

People

(Reporter: dbaron, Unassigned)

Tracking

(Blocks: 2 bugs, {dev-doc-needed, site-compat})

Trunk
dev-doc-needed, site-compat
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox53 affected)

Details

(URL)

Attachments

(2 attachments)

We should unprefix the keywords at https://drafts.csswg.org/css-sizing/#width-height-keywords .  This includes renaming -moz-available to fill (different name).

See also bug 1312588.
Keywords: dev-doc-needed, site-compat
Created attachment 8820716 [details]
image fill test

It seems our -moz-available isn't compatible with Chrome's -webkit-fill-available.
(Firefox overflows in the vertical dimension when the window is wide,
Chrome does not.
Firefox preserves the image ratio when the window is narrow, Chrome does not.)
(In reply to Mats Palmgren (:mats) from comment #1)
> It seems our -moz-available isn't compatible with Chrome's
> -webkit-fill-available.
> (Firefox overflows in the vertical dimension when the window is wide,
> Chrome does not.
> Firefox preserves the image ratio when the window is narrow, Chrome does
> not.)

Maybe the difference in this test is that they support the keyword for the block-size dimension and we only support it for inline-size?
Created attachment 8820886 [details]
image fill test, one axis only

It looks like Chrome support it in both dimensions, but the ratio-preservation
is wrong in some cases and they also overflow in some cases.  But, they do
preserve the ratio here when it's only specified in one axis, unlike
the first testcase that has 'fill' in both axes. 

It seems Gecko only support 'fill' in the inline-direction though.
We should probably not unprefix this before we support it in both.

Comment 4

10 months ago
fill keyword now renamed stretch

https://drafts.csswg.org/css-sizing-3/#valdef-width-stretch
https://drafts.csswg.org/css-sizing-3/#changes

Updated

5 months ago
Priority: -- → P3
You need to log in before you can comment on or make changes to this bug.