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

Add compat data for Slotable api #1232

Merged
merged 6 commits into from Mar 7, 2018
Merged

Add compat data for Slotable api #1232

merged 6 commits into from Mar 7, 2018

Conversation

luciancic
Copy link

No description provided.

@teoli2003 teoli2003 added the data:api 🐇 Compat data for Web APIs. https://developer.mozilla.org/docs/Web/API label Mar 1, 2018
@Elchi3 Elchi3 added this to In Progress / Review Needed in Migration of compat data from MDN pages Mar 1, 2018
Copy link
Contributor

@ExE-Boss ExE-Boss left a comment

Choose a reason for hiding this comment

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

Corrected Firefox support information.

"version_added": null
},
"firefox": {
"version_added": true
Copy link
Contributor

Choose a reason for hiding this comment

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

This should be:

{
  "version_added": "58",
  "flags": [
    {
      "type": "preference",
      "name": "dom.webcomponents.enabled",
      "value_to_set": "true"
    }
  ]
}

"version_added": true
},
"firefox_android": {
"version_added": true
Copy link
Contributor

Choose a reason for hiding this comment

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

ditto

]
},
"firefox_android": {
"version_added": true
Copy link
Contributor

Choose a reason for hiding this comment

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

You missed the information for Firefox for Android, this should be the same as in Firefox for Desktop.

Copy link
Author

Choose a reason for hiding this comment

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

Right, my bad

Copy link
Collaborator

@chrisdavidmills chrisdavidmills left a comment

Choose a reason for hiding this comment

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

The compat data on here needs updating, as the old-fashioned table on the Slotable MDN page isn't quite right.

Since this was implemented in most browsers as part of the same effort as the rest of the Shadow DOM v1 spec, I'd fully expect this to have the same support data as HTMLSlotElement. Copy the support data from https://github.com/chrisdavidmills/browser-compat-data/blob/dc6d108eb722a254774a9be7dcfe1f2463ee5e90/api/HTMLSlotElement.json, for both the Interface and its property.

Thanks!

@luciancic
Copy link
Author

Sorry for the messiness, I forgot to pull before pushing. And thanks for the explanation; I'm still learning and it's a huge help.

@chrisdavidmills
Copy link
Collaborator

No worries! It is a complex process; it took me a while to get used to as well. And the Web components support data is not the simplest. Thanks for the contribution ;-)

@Elchi3 Elchi3 merged commit 9a837de into mdn:master Mar 7, 2018
Migration of compat data from MDN pages automation moved this from In Progress / Review Needed to MDN page updates needed Mar 7, 2018
@Elchi3 Elchi3 moved this from MDN page updates needed to Done in Migration of compat data from MDN pages Mar 9, 2018
dontcallmedom pushed a commit to dontcallmedom/browser-compat-data that referenced this pull request Mar 17, 2018
mlbrgl pushed a commit to mlbrgl/browser-compat-data that referenced this pull request Mar 29, 2018
@luciancic luciancic deleted the slotable branch April 7, 2018 18:04
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
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

6 participants