1598151 - [motion-1] Ship offset-path:ray()

Status: RESOLVED FIXED

(Core :: CSS Transitions and Animations, enhancement, P3)

Comment 1

[Tom Ritter [:tjr] (OOTO until 4/30 at least)]

layout.css.motion-path-ray.enabled has been enabled in Nightly for several years at this point I believe...

Comment 2

[Sebastian Zartner [:sebo]]

(In reply to Tom Ritter [:tjr] (sec-approvals paused until after release) from comment #1)

layout.css.motion-path-ray.enabled has been enabled in Nightly for several years at this point I believe...

Yes, but there are several open issues blocking shipping. I've added the ones I saw in the main tracking bug now. Could be that there are others.
Maybe Boris can decide how to proceed here.

Sebastian

Comment 3

[Boris Chiou [:boris]]

(In reply to Sebastian Zartner [:sebo] from comment #2)

(In reply to Tom Ritter [:tjr] (sec-approvals paused until after release) from comment #1)

layout.css.motion-path-ray.enabled has been enabled in Nightly for several years at this point I believe...

Yes, but there are several open issues blocking shipping. I've added the ones I saw in the main tracking bug now. Could be that there are others.
Maybe Boris can decide how to proceed here.

Sebastian

Thanks for adding these dependencies. I didn't finish this because of the spec issue in Bug 1581237, and at that moment, only Gecko and Chrome implemented ray() function (and I jumped to other topics). However, it seems WebKit had implemented ray() last year (WebKit Bug 233153), and so perhaps it's time to keep working on this (i.e. finish the dependencies and ship ray()). This is not in the backlog of layout team now, so I'd like to mention this in the layout meeting or my 1-on-1 with Frank.

Comment 4

[Boris Chiou [:boris]]

Attachment: Bug 1598151 - Ship offset-path: ray().

The spec issue for offset-path: ray() has been resolved, and we match the spec.
Also, other browsers have shipped this feaure already.

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/12965d3a5b12
Ship offset-path: ray(). r=jwatt

Comment 6

[Iulian Moraru]

https://hg.mozilla.org/mozilla-central/rev/12965d3a5b12

Comment 7

[Donal Meehan [:dmeehan]]

:jwatt sorry for the ping Boris has a PTO until Jan 15 status, you might be able to help with context.
Could you consider nominating this for a release note? (Process info)

Comment 8

[Boris Chiou [:boris]]

Release Note Request (optional, but appreciated)
[Why is this notable]: Extension to offset-path feature
[Affects Firefox for Android]: yes
[Suggested wording]: Support ray() for offset-path CSS property on all channels
[Links (documentation, blog post, etc)]: https://groups.google.com/a/mozilla.org/g/dev-platform/c/TNdVbrT-j8o or this bug

Comment 9

[Donal Meehan [:dmeehan]]

Thanks, added to the Fx122 nightly release notes, please allow 30 minutes for the site to update.
Keeping the relnote-firefox flag as ? to keep it on the radar for inclusion in the final Fx122 release notes

Comment 10

[Dipika]

MDN doc updates corresponding to this change can be tracked via https://github.com/mdn/content/issues/31099