Skip to content

Conversation

@frittate
Copy link
Contributor

@Elchi3 Elchi3 added the data:html Compat data for HTML elements. https://developer.mozilla.org/docs/Web/HTML label Feb 15, 2018
@Elchi3
Copy link
Member

Elchi3 commented Feb 15, 2018

Hey, thanks for your pull request! 💯

If you want to validate the JSON locally, you can also do so using npm test. See also our contributing file. If you have any more questions, feel free to ask them here as well.

@frittate
Copy link
Contributor Author

frittate commented Feb 15, 2018

OK, got it. I get the same error: global_attributes should NOT have additional properties.

This is due to the file format being html.global_attributes having all attributes listed below.
How should the file be structured instead?

"chrome": {
"version_added": true,
"version_removed": "52"
},
Copy link
Contributor

Choose a reason for hiding this comment

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

chrome_android and webview_android should match this.

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

Choose a reason for hiding this comment

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

"mdn_url": "https://developer.mozilla.org/docs/Web/HTML/Global_attributes/spellcheck",
"support": {
"chrome": {
"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.

"partial_implementation": false
},
"webview_android": {
"version_added": "4.4",
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 47. Webview version numbers follow Chromium, not Android.

}
}
},
"data-*": {
Copy link
Member

Choose a reason for hiding this comment

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

This is not a valid identifier unfortunately. (the error message is not helpful, so you couldn't know this.
"data-*": {

Better would be:

"data_attributes": {
  "__compat": {
    "description": "<code>data-*</code> attributes",
    "mdn_url": "...",
    "support": {

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.

Thanks so much for your work! I've fixed up the minor issues for you. Let's merge this! 🎉

@Elchi3 Elchi3 merged commit 06a0edf into mdn:master Feb 28, 2018
dontcallmedom pushed a commit to dontcallmedom/browser-compat-data that referenced this pull request Mar 17, 2018
queengooborg added a commit to queengooborg/browser-compat-data that referenced this pull request Jun 18, 2024
…support

This PR updates and corrects version values for all browsers for the `title.multi-line-support` member of the `global_attributes` HTML feature. This sets the feature(s) to a version range based upon the date that the feature was added to BCD with the intent of replacing `true` values with ranged values to eliminate `true` values from BCD.

Commit/PR Adding the Feature: mdn#1089
Elchi3 pushed a commit that referenced this pull request Jun 18, 2024
…support (#23424)

This PR updates and corrects version values for all browsers for the `title.multi-line-support` member of the `global_attributes` HTML feature. This sets the feature(s) to a version range based upon the date that the feature was added to BCD with the intent of replacing `true` values with ranged values to eliminate `true` values from BCD.

Commit/PR Adding the Feature: #1089
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

data:html Compat data for HTML elements. https://developer.mozilla.org/docs/Web/HTML

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants