1816964 - `linear()` easing does not conform to spec for blank and 0 values

Status: RESOLVED FIXED

(Core :: CSS Transitions and Animations, defect)

Description

[Oliver Medhurst [:canadahonk]]

As per the spec, linear() and linear(0) should not be valid as it states:

If there are less than two items in stopList, then return failure.

WPT tests seem to also be wrong in regards to this?

See also:

Comment 1

[David Shin[:dshin]]

Thanks for the report - you're right, reading the latest spec, 0/1 argument to linear() should no longer be considered valid.
(For context, the implementation is based on the earlier proposal.)

Comment 2

[Oliver Medhurst [:canadahonk]]

I was just reading, thanks for the confirmation. I am happy to make a patch which fixes this and WPT tests if that is good with others?

Comment 3

[David Shin[:dshin]]

Attachment: Bug 1816964 - `linear()` easing function should not consider less than 2 linear stop arguments valid. r=#layout-reviewers

Latest spec no longer handles linear() == linear and linear(<single value>) == <single value>.

Comment 4

[David Shin[:dshin]]

My bad - did not notice bug 1816964 comment 2.
Still, thanks for looking into this.

Comment 5

[Oliver Medhurst [:canadahonk]]

No worries! Thanks for the fix.

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/d90009e8fa12
`linear()` easing function should not consider less than 2 linear stop arguments valid. r=emilio

Comment 7

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

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

Comment 8

[Natalia Csoregi [:nataliaCs]]

https://hg.mozilla.org/mozilla-central/rev/d90009e8fa12

Comment 9

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

Upstream PR merged by jgraham

Comment 10

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/9cdbaa867dec
[wpt PR 38569] - [Gecko Bug 1816964] linear()` easing function should not consider less than 2 linear stop arguments valid., a=testonly

Comment 11

[Noemi Erli[:noemi_erli]]

https://hg.mozilla.org/mozilla-central/rev/9cdbaa867dec