Update Safari versions for Animation API #17021
Conversation
This PR updates and corrects the real values for Safari (Desktop and iOS/iPadOS) for the `Animation` API, based upon results from the [mdn-bcd-collector](https://mdn-bcd-collector.appspot.com) project (v6.0.8). Tests Used: https://mdn-bcd-collector.appspot.com/tests/api/Animation _Check out the [collector's guide on how to review this PR](https://github.com/foolip/mdn-bcd-collector#reviewing-bcd-changes)._ Note: due to issues with `update-bcd`, these changes were performed manually by comparing the results output for Safari 10.0 and 10.1.
queengooborg
added
the
data:api
There was a problem hiding this comment.
This doesn't match the data for https://developer.mozilla.org/en-US/docs/Web/API/Element/animate. Maybe the interface was exposed before it could actually be used for anything?
Can you investigate those issues? Given WebKit/WebKit@b206b8c + WebKit/WebKit@28472b3 I suspect that these interfaces were exposed for one release by accident. That would lead to the collector bailing here: I think fixing this issue will be necessary for openwebdocs/project#85, or at least we have to be able to list the errors even if the collector can't fix them. |
|
Ah, yep, that's what's going on. Because two of the properties were supported in that early version of Safari as well, the collector was setting them to Safari 10.1. Between 10.1 and 13.1, they all returned |
This PR updates and corrects the real values for Safari (Desktop and iOS/iPadOS) for the
AnimationAPI, based upon results from the mdn-bcd-collector project (v6.0.8).Tests Used: https://mdn-bcd-collector.appspot.com/tests/api/Animation
Check out the collector's guide on how to review this PR.
Note: due to issues with
update-bcd, these changes were performed manually by comparing the results output for Safari 10.0 and 10.1.