Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adds compat data for AnimationEvent #1017

Merged
merged 5 commits into from
Feb 16, 2018
Merged

Conversation

bennypowers
Copy link
Contributor

@bennypowers bennypowers commented Feb 11, 2018

Fixes #965

@bennypowers
Copy link
Contributor Author

bennypowers commented Feb 11, 2018

Fixes #965

@Elchi3 Elchi3 added the data:api 🐇 Compat data for Web APIs. https://developer.mozilla.org/docs/Web/API label Feb 12, 2018
Copy link
Member

@Elchi3 Elchi3 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

},
"chrome": [
{
"version_added": "1",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please swap this around, the more relevant version is the unprefixed one. (the first element will be rendered as the main compat cell in the MDN tables, other version ranges will be put into the expandable details.)

        "chrome": [
            {
              "version_added": "43"
            }
            {
              "version_added": "1",
              "prefix": "webkit"
            },
          ],

"mdn_url": "https://developer.mozilla.org/docs/Web/API/AnimationEvent",
"support": {
"webview_android": {
"version_added": null,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Change to true.

"support": {
"webview_android": {
"version_added": null,
"prefix": "webkit"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unprefixed in 43.

},
"chrome": [
{
"version_added": "43"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this how we note when something was unprefixed? @Elchi3

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, the array indicates support ranges.
So, there is a support range starting from version 43 where this is fully available and unprefixed.
Then there is a second support ranges where it was available (version not known) and prefixed.
The rendered tables take the first array element as the one to render in the main cell. The other support ranges will be treated as secondary and put into the expandable info. See e.g. https://developer.mozilla.org/en-US/docs/Web/API/CanvasRenderingContext2D/imageSmoothingEnabled#Browser_compatibility

}
],
"chrome_android": {
"version_added": "43"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

true for original version added. "43" is for when the prefix was removed.

"mdn_url": "https://developer.mozilla.org/docs/Web/API/AnimationEvent/AnimationEvent",
"support": {
"webview_android": {
"version_added": false
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should be true and prefix removed in "43". Repeat for Chrome and Chrome Android.

"version_added": true
},
"chrome_android": {
"version_added": "43"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All three versions of Chrome should have the same data as for the constructor.

"version_added": "4"
},
"webview_android": {
"version_added": null
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should be true.

"version_removed": "23"
},
"opera_android": {
"version_added": null
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

true.

@Elchi3
Copy link
Member

Elchi3 commented Feb 16, 2018

Well done! We're almost there.
I think there is one more member of this interface that I've missed in my last review: https://developer.mozilla.org/en-US/docs/Web/API/AnimationEvent/elapsedTime

Copy link
Member

@Elchi3 Elchi3 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
data:api 🐇 Compat data for Web APIs. https://developer.mozilla.org/docs/Web/API
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants