Update parameter entries to match guidelines#11080
Conversation
ddbeck
left a comment
There was a problem hiding this comment.
Thank you! I checked the content and there was only one change needed (which I took care of, see line comments for details). I also spotted one rename that might not be necessary.
api/Document.json
Outdated
| } | ||
| }, | ||
| "deep_optional": { | ||
| "deep_parameter_optional": { |
There was a problem hiding this comment.
I'm all but certain this is an irrelevant feature. My impression is that deep was required when it was first introduced, but isn't now. I guess it's possible that EdgeHTML required, but I doubt it, given how thin this data is anyway. I'd rather see this one go away entirely.
There was a problem hiding this comment.
I'll revert this change and send a separate PR. Since the Firefox versions are set this smells a lot like a "we changed something and should document it" entry that stops being useful very soon after the Firefox release.
See https://github.com/mdn/browser-compat-data/blob/main/docs/data-guidelines.md#parameters-and-parameter-object-features