Bug 229135 – Make most pseudo elements with display: contents; compute display value to inline by default

• Component: CSS
• Status: NEW

[Tim Nguyen (:ntim)]

Except ::before/::after/::file-selector-button where display: contents should just work.

Except ::backdrop where it should compute to block (though this will be handled along with the backdrop + top layer implementation).

[Tim Nguyen (:ntim)]

See also: https://github.com/w3c/csswg-drafts/issues/6518

[Radar WebKit Bug Importer]

<rdar://problem/82239251>

[Tim Nguyen (:ntim)]

Created attachment 441352 [details]
Patch

[Antti Koivisto]

this needs a test

[Tim Nguyen (:ntim)]

(In reply to Antti Koivisto from comment #4)
> this needs a test

Do you know where this case is ever reached? I'm tempted to remove the block if it isn't reached.

[Antti Koivisto]

> Do you know where this case is ever reached? I'm tempted to remove the block
> if it isn't reached.

No, but I would expect that the person who wants to change the code does.

[Tim Nguyen (:ntim)]

Will look into this when I have more time.

[Ahmad Saleem]

is it now Not Applicable since Gecko has bug filed against it? https://bugzilla.mozilla.org/show_bug.cgi?id=1418138

This part - style.setEffectiveDisplay(DisplayType::Inline);

Don't know about: if (isInTopLayerOrBackdrop(style, m_element) || m_document.documentElement() == m_element)