Intent to ship: multi-keyword values on the CSS 'display' property

Summary:
Add support for multi-keyword values on the CSS 'display' property.
The spec splits this property into three parts: an outside part,
an inside part, and a list-item part so the author can specify them
separately.  This first patch set adds support for the new syntax
only, but no new box types (I'll add those separately in a bit).
So, this bug just adds a bunch of synonyms for exist values we
already support, so we can now write "block flow list-item" or
"block list-item" etc instead of just "list-item".  Keywords can
be given in arbitrary order, so there are now a lot of valid
permutations for the same computed value.  The values are
serialized to their shortest form as usual though.

Bug: https://bugzilla.mozilla.org/show_bug.cgi?id=1038294

Standard: https://drafts.csswg.org/css-display/#the-display-properties

Platform coverage: All

Estimated or target release: v70

Preference: None

DevTools: No change

Tests: WPT and other tests were added.

Other browsers: Other UAs don't support this yet, AFAIK.


/Mats
0
Mats
8/14/2019 4:37:48 PM
mozilla.dev.platform 6545 articles. 0 followers. Post Follow

2 Replies
8 Views

Similar Articles

[PageSpeed] 22

Summary:
Add support for multi-keyword values on the CSS 'display' property.
The spec splits this property into three parts: an outside part,
an inside part, and a list-item part so the author can specify them
separately.  This first patch set adds support for the new syntax
only, but no new box types (I'll add those separately in a bit).
So, this bug just adds a bunch of synonyms for exist values we
already support, so we can now write "block flow list-item" or
"block list-item" etc instead of just "list-item".  Keywords can
be given in arbitrary order, so there are now a lot of valid
permutations for the same computed value.  The values are
serialized to their shortest form as usual though.

Bug: https://bugzilla.mozilla.org/show_bug.cgi?id=1038294

Standard: https://drafts.csswg.org/css-display/#the-display-properties

Platform coverage: All

Estimated or target release: v70

Preference: None

DevTools: No change

Tests: WPT and other tests were added.

Other browsers: Other UAs don't support this yet, AFAIK.


/Mats
0
Mats
8/14/2019 4:37:48 PM
On 8/14/19 12:37 PM, Mats Palmgren wrote:
> This first patch set adds support for the new syntax
> only, but no new box types (I'll add those separately in a bit).

In general, it seems like we should think about how to integrate this 
stuff into layout in a better way than anonymous boxes everywhere.  In 
particular, we may want to consider changing nsIFrame such that we have 
it point to a "how I look to my parent" class and a "how I manage my 
kids or paint myself" class and be able to mix and match those.  That's 
out of scope for this intent, but if we plan to start adding various 
implementations of multiple keywords for display this seems like it 
would be the way to go about it...

> Other browsers: Other UAs don't support this yet, AFAIK.

Do they plan to?

-Boris
0
Boris
8/14/2019 4:52:56 PM
Reply: