Closed Bug 1598151 Opened 5 years ago Closed 4 months ago

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

Categories

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

enhancement

Tracking

()

RESOLVED FIXED
122 Branch
Tracking Status
relnote-firefox --- 122+
firefox122 --- fixed

People

(Reporter: boris, Assigned: boris)

References

(Blocks 2 open bugs, )

Details

(4 keywords)

Attachments

(1 file)

No description provided.
Keywords: dev-doc-needed

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

(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

Depends on: 1581237, 1590542, 1579294
Flags: needinfo?(boris.chiou)

(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.

Flags: needinfo?(boris.chiou)
Severity: normal normal → S3 S3
Depends on: 1820070
Depends on: 1820071
Depends on: 1821450
Depends on: 1836831
No longer depends on: 1836831

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

Assignee: nobody → boris.chiou
Status: NEW → ASSIGNED
Pushed by bchiou@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/12965d3a5b12
Ship offset-path: ray(). r=jwatt
Status: ASSIGNED → RESOLVED
Closed: 4 months ago
Resolution: --- → FIXED
Target Milestone: --- → 122 Branch

: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)

Flags: needinfo?(jwatt)

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

relnote-firefox: --- → ?
Flags: needinfo?(jwatt)

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

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

You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: