facebook / flow

Adds static typing to JavaScript to improve developer productivity and code quality.
https://flow.org/
MIT License
22.07k stars 1.85k forks source link

Missing DOM HTML elements #3703

Open mislav opened 7 years ago

mislav commented 7 years ago

These HTML element classes are defined in Google Chrome but missing from dom.js definitions:

This list was generated with:

Object.getOwnPropertyNames(window).filter(n => /^HTML/.test(n)).join("\n")
jschfflr commented 7 years ago

@mislav I implemented the HTMLObjectElement class. Should I open a pull request?

mislav commented 7 years ago

@ghodir Always feel free to open a pull request!

Keep in mind that adding some tests for this could be useful. See existing tests/dom/*.js files. Re-run tests with make && ./runtests.sh bin/flow dom. Record new test expectations with ./runtests.sh -r bin/flow dom.

mroch commented 6 years ago

https://github.com/facebook/flow/commit/7b971fbd7b0ec8c66613459eac6cfd31a0237b26 added a bunch, I updated the checklist above

mislav commented 6 years ago

@mroch Fantastic; thank you

romeovs commented 5 years ago

HTMLSummaryElement is not on this list! Should I open a new issue or can we add it to this one?

mislav commented 5 years ago

@romeovs The spec suggests that <summary> does not have its own HTMLSummaryElement interface and that it instead uses HTMLElement https://html.spec.whatwg.org/multipage/interactive-elements.html#the-summary-element

Similarly, Google Chrome does not seem to have HTMLSummaryElement defined either.