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 missing features for HTMLInputElement #7592

Closed
wants to merge 3 commits into from

Conversation

queengooborg
Copy link
Collaborator

This PR is a part of a project to add missing interfaces and interface features to BCD that are from a spec in Editor's Draft or more and is supported in at least one browser. This particular PR adds missing features, populating the results using data from the mdn-bcd-collector project (v1.1.6), for the HTMLInputElement API.

Spec: https://html.spec.whatwg.org/multipage/semantics.html#htmlinputelement
IDL: https://github.com/w3c/webref/blob/master/ed/idl/html.idl

Notice: the diff looks pretty weird for this one due to the similarities in the results between all of the features. This PR doesn't modifying any existing features, however.

@github-actions github-actions bot added the data:api 🐇 Compat data for Web APIs. https://developer.mozilla.org/docs/Web/API label Dec 6, 2020
@foolip
Copy link
Collaborator

foolip commented Dec 8, 2020

This PR is huge, and will also require checking that reflected IDL attributes have their content attribute data updated. @vinyldarkscratch how about splitting this into smaller pieces, where the diff of each piece becomes more managable?

Copy link
Collaborator

@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.

Took another look, but this is still very large and difficult to review. Splitting it into smaller bits in a way that avoids moves becoming apparent changes would really help.

@@ -95,33 +94,32 @@
}
}
},
"autofocus": {
"align": {
Copy link
Collaborator

Choose a reason for hiding this comment

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

align is listed in https://html.spec.whatwg.org/multipage/obsolete.html#HTMLInputElement-partial and should be deprecated. (But I can't make suggestions in that part of the diff.)

"experimental": false,
"standard_track": true
"standard_track": true,
"deprecated": false
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
"deprecated": false
"deprecated": true

https://html.spec.whatwg.org/multipage/obsolete.html#HTMLInputElement-partial

@@ -191,47 +188,44 @@
}
}
},
"files": {
"autocapitalize": {
Copy link
Collaborator

Choose a reason for hiding this comment

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

autocapitalize shouldn't be added: #7519

@queengooborg
Copy link
Collaborator Author

I separated this PR into three other PRs, see #7647, #7648, and #7649.

@queengooborg queengooborg deleted the api/HTMLInputElement branch December 11, 2020 13:09
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