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 HTMLInputElement showPicker() #14444

Merged
merged 9 commits into from
Jan 7, 2022
Merged

Add HTMLInputElement showPicker() #14444

merged 9 commits into from
Jan 7, 2022

Conversation

beaufortfrancois
Copy link
Contributor

This PR adds HTMLInputElement.showPicker()

See related PR at mdn/content#11753

@github-actions github-actions bot added the data:api 🐇 Compat data for Web APIs. https://developer.mozilla.org/docs/Web/API label Jan 5, 2022
@beaufortfrancois
Copy link
Contributor Author

Since mdn/content#11753 got merged, https://developer.mozilla.org/en-US/docs/Web/API/HTMLInputElement/showPicker is now live.
Hopefully Chrome 99 will be added to the list of supported versions soon.

@beaufortfrancois
Copy link
Contributor Author

@foolip Can you review by any chance?

api/HTMLInputElement.json Outdated Show resolved Hide resolved
api/HTMLInputElement.json Outdated Show resolved Hide resolved
api/HTMLInputElement.json Outdated Show resolved Hide resolved
api/HTMLInputElement.json Outdated Show resolved Hide resolved
@foolip
Copy link
Collaborator

foolip commented Jan 7, 2022

@beaufortfrancois just to set expectations, I don't always post a review with suggestions 10 seconds after being asked 😄

beaufortfrancois and others added 4 commits January 7, 2022 10:41
Co-authored-by: Philip Jägenstedt <philip@foolip.org>
Co-authored-by: Philip Jägenstedt <philip@foolip.org>
Co-authored-by: Philip Jägenstedt <philip@foolip.org>
Co-authored-by: Philip Jägenstedt <philip@foolip.org>
@beaufortfrancois
Copy link
Contributor Author

@beaufortfrancois just to set expectations, I don't always post a review with suggestions 10 seconds after being asked 😄

Loved it though ;)

@beaufortfrancois
Copy link
Contributor Author

@foolip Argh

api.HTMLInputElement.showPicker - version_added: "99" is NOT a valid version number for edge

Shall we add edge version number or wait?

@beaufortfrancois
Copy link
Contributor Author

@foolip I've sent #14456 for review to fix this.

@foolip
Copy link
Collaborator

foolip commented Jan 7, 2022

@beaufortfrancois a rebase of this should fix the CI failure now.

@beaufortfrancois
Copy link
Contributor Author

@beaufortfrancois a rebase of this should fix the CI failure now.

Done!

@foolip foolip merged commit 07bff73 into mdn:main Jan 7, 2022
jpmedley pushed a commit that referenced this pull request Jan 10, 2022
Co-authored-by: Philip Jägenstedt <philip@foolip.org>
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.

2 participants