Re: [csswg-drafts] [web-animations-2] Update OptionalEffectTiming.duration to accept CSSNumericValue (#8949)

_I initially reported this with TypeScript because I thought it was an issue there, not a spec issue._

For context, our use case is reading the current timing and adjusting that.
For this purpose it is important that these values can "round trip".

```ts
function reduceDuration( timing: EffectTiming ): OptionalEffectTiming {
 if ( 'auto' === timing.duration ) {
  timing.duration = 1;
 } else {
  timing.duration = timing.duration / 1000;
 }

 return timing;
}


keyframeEffect.updateTiming(
 reduceDuration(keyframeEffect.getTiming())
);
```

This has been used in production for years.

With the updated typings it no longer works, but I now also fear that the implementations might change so that the actual code might also break in the future.

-- 
GitHub Notification of comment by romainmenke
Please view or discuss this issue at https://github.com/w3c/csswg-drafts/issues/8949#issuecomment-1588819862 using your GitHub account


-- 
Sent via github-notify-ml as configured in https://github.com/w3c/github-notify-ml-config

Received on Tuesday, 13 June 2023 08:35:14 UTC