-
Notifications
You must be signed in to change notification settings - Fork 2k
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
"Only supported for HTMLDocument" notes inconsistently applied #10682
Comments
For what it's worth, here's a slightly updated version of the Live DOM Viewer test with ✅ and ❌ instead of "true" and "false" to make it more easy to spot the differences. Edit: And here's a test that only shows the inconsistencies, i.e. omitting what's supported in all or none: https://software.hixie.ch/utilities/js/live-dom-viewer/?saved=13374 |
I have gathered data in this repo. For example, this is the summary for Firefox: https://github.com/caugner/document-members/blob/main/issues/firefox.txt Here's what Firefox version fixed which members:
|
This is the summary for Chrome: https://github.com/caugner/document-members/blob/main/issues/chrome.txt Here's what Chrome version fixed which members:
|
Here's a better overview for Chrome, Firefox and Safari: https://github.com/caugner/document-members/blob/main/SUMMARY.md |
#6839 fixed a bunch of entries for things that have been moved from
HTMLDocument
toDocument
, but there are still lots of APIs which were once onHTMLDocument
which haven't gotten this treatment.https://software.hixie.ch/utilities/js/live-dom-viewer/?saved=9330 is a test for all the entries that are currently in Document.json. As an example
document.head
seems to have been moved fromHTMLDocument
toDocument
in Firefox 61.This is fairly well suited to systematic testing, but https://mdn-bcd-collector.appspot.com/ doesn't generate the variant tests that would be needed to do this. It's also doubtful that it's worth automating, since it would be fixed once and then not need any further testing.
The text was updated successfully, but these errors were encountered: