Skip to content

Add/update Opera Presto data for Document API#6835

Merged
foolip merged 5 commits intomdn:masterfrom
queengooborg:api/Document-opera
Oct 9, 2020
Merged

Add/update Opera Presto data for Document API#6835
foolip merged 5 commits intomdn:masterfrom
queengooborg:api/Document-opera

Conversation

@queengooborg
Copy link
Copy Markdown
Contributor

This PR updates the Opera (Presto) data for the Document API based upon manual testing to achieve more real data for the Document API. Some data is mirrored from Chrome.

Tests used: https://mdn-bcd-collector.appspot.com/tests/api/Document

@github-actions github-actions bot added the data:api Compat data for Web APIs. https://developer.mozilla.org/docs/Web/API label Oct 5, 2020
@foolip
Copy link
Copy Markdown
Contributor

foolip commented Oct 8, 2020

@vinyldarkscratch some createProcessingInstruction changes for other browsers came along for the ride, which I've reverted.

Copy link
Copy Markdown
Contributor

@foolip foolip left a comment

Choose a reason for hiding this comment

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

Good stuff, I'd just like to separate out the non-Presto stuff.

},
"opera": {
"version_added": null
"version_added": false
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks like we should delete the api.Document.domConfig entry :)

@foolip foolip merged commit 12aa3cd into mdn:master Oct 9, 2020
@queengooborg queengooborg deleted the api/Document-opera branch October 10, 2020 19:13
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