1820071 - Make <ray-size> optional and its default value is 'closest-side'

Status: RESOLVED FIXED

(Core :: Layout, enhancement)

Description

[Boris Chiou [:boris]]

Due to the spec change, <ray-size> is optional now. We have to update the parser.

Comment 1

[Boris Chiou [:boris]]

Attachment: Bug 1820071 - Make <ray-size> optional and default to 'closest-side'.

Per the spec update, the new syntax is:
ray() = ray( <angle> && <ray-size>? && contain? )
And for <ray-size>:
"If no <ray-size> is specified it defaults to closest-side."

So <ray-size> is optional and we omit it if it's default value, for
serialization.

By the way, offset=* properties are supported only in Gecko, so we don't
need a servo function to check the preference.

Comment 2

[Boris Chiou [:boris]]

Note: we need to update MDN for offset-path syntax (https://developer.mozilla.org/en-US/docs/Web/CSS/offset-path).

Reference spec link: https://drafts.fxtf.org/motion/#ray-function.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/6c84f9f0b88d
Make <ray-size> optional and default to 'closest-side'. r=emilio

Comment 4

[Web Platform Test Sync Bot (Matrix: #interop:mozilla.org)]

Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/38840 for changes under testing/web-platform/tests

Comment 5

[Noemi Erli[:noemi_erli]]

https://hg.mozilla.org/mozilla-central/rev/6c84f9f0b88d

Comment 6

[Web Platform Test Sync Bot (Matrix: #interop:mozilla.org)]

Upstream PR merged by moz-wptsync-bot

Comment 7

[Dipika]

The doc updates on MDN for this feature can be tracked via this issue tracker: https://github.com/mdn/content/issues/25358.

Thanks a lot, Boris, for all your help understanding this feature behavior!