Closed Bug 1429301 Opened 5 years ago Closed 4 years ago

Implement offset-rotate for motion path

Categories

(Core :: CSS Parsing and Computation, enhancement, P3)

enhancement

Tracking

()

RESOLVED FIXED
mozilla69
Tracking Status
firefox69 --- fixed

People

(Reporter: boris, Assigned: boris)

References

(Blocks 1 open bug, )

Details

(Keywords: dev-doc-complete)

Attachments

(1 file, 2 obsolete files)

We implement offset-rotate [1] in this bug. offset-rotate defines the orientation of the box while positioning along the offset-path.

offset-rotate is an animatable property, but for simplicity, we could make it non-animatable for now.

Note:
The computed value of offset-rotate doesn't need to store "reverse" because "reverse <deg>" is immediately converted to "auto <180+deg>", so the interpolation and property equality code never needs to consider "reverse".

[1] https://drafts.fxtf.org/motion-1/#offset-rotate-property
Blocks: 1429303
Assignee: boris.chiou → nobody
Status: ASSIGNED → NEW
Priority: -- → P3
Assignee: nobody → boris.chiou
Attachment #9066898 - Attachment description: spec issue, #340 → https://github.com/w3c/fxtf-drafts/issues/340
Attachment #9066898 - Attachment is obsolete: true
Attachment #9066843 - Attachment description: Bug 1429301 - Part 1: Implement offset-rotate in style system. → Bug 1429301 - Implement offset-rotate.
Attachment #9066844 - Attachment is obsolete: true
Pushed by bchiou@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/ae71e496d587
Implement offset-rotate. r=emilio
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/16987 for changes under testing/web-platform/tests
Upstream web-platform-tests status checks passed, PR will merge once commit reaches central.
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla69

The docs are finished for Motion Path, for now. See https://github.com/mdn/sprints/issues/1961#issuecomment-526318438 for the specifics of what I've done.

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