Closed Bug 1670155 Opened 4 years ago Closed 3 years ago

Implement interpolation for <ratio>

Categories

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

defect

Tracking

()

RESOLVED FIXED
88 Branch
Tracking Status
firefox88 --- fixed

People

(Reporter: boris, Assigned: boris)

References

()

Details

Attachments

(2 files)

The discussion is in https://github.com/w3c/csswg-drafts/issues/4953.

The conclusion is

  • Resolve the ratio into a single number, and interpolate that value. (Aka turn it into X / 1 form.)
Assignee: nobody → boris.chiou

The animation type of aspect-ratio is discrete so perhaps we don't have to fix this bug now.

Summary: Convert <ratio> to an equivalent form with a ''1'' as its second value for interpolation → Implement interpolation for <ratio>

Based on https://drafts.csswg.org/css-values/#ratios, <ratio> should be
a general types in css values, and now the media query and the position use
this type, so let's move it into the independent files.

I also update the wpt becasue it seems the original one lets <ratio>
support the addition. However, the spec says "Addition of <ratio>s is not
possible".

Attachment #9204997 - Attachment description: Bug 1670155 - Implement Animate trait for <ratio>. → Bug 1670155 - Implement Animate trait and ComputeSquaredDistance for <ratio>.
Attachment #9204997 - Attachment description: Bug 1670155 - Implement Animate trait and ComputeSquaredDistance for <ratio>. → Bug 1670155 - Implement Animate trait and ComputeSquaredDistance trait for <ratio>.
Pushed by bchiou@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/0727fcb31003
Move Ratio into independent files. r=emilio
https://hg.mozilla.org/integration/autoland/rev/5cfe91fdb8ff
Implement Animate trait and ComputeSquaredDistance trait for <ratio>. r=emilio
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/27773 for changes under testing/web-platform/tests
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 88 Branch
Upstream PR merged by moz-wptsync-bot
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: