Skip to content

feat(react-dom): Add enterKeyHint#18634

Merged
gaearon merged 1 commit into
react:masterfrom
eps1lon:feat/enterkeyhint
Jul 27, 2020
Merged

feat(react-dom): Add enterKeyHint#18634
gaearon merged 1 commit into
react:masterfrom
eps1lon:feat/enterkeyhint

Conversation

@eps1lon

@eps1lon eps1lon commented Apr 16, 2020

Copy link
Copy Markdown
Collaborator

Summary

-<input enterkeyhint="search" />
+<input enterKeyHint="search" />

Test Plan

Ran /fixtures/attribute-behavior locally. Results available under https://drive.google.com/open?id=1j5PJgc-NcKXsGZh8GqBfHXHfdEwd9ii_.

Note: Should I update the fixture for inputMode as well? It is still reading from the attribute but the property is supported as of chrome 66.

https://github.com/facebook/react/blob/8c75389319d479dc5b2f2cf4b2c7084549913084/fixtures/attribute-behavior/src/attributes.js#L955

Codesandbox using this PR: https://codesandbox.io/s/enterkeyhint-6l5t3

@codesandbox-ci

Copy link
Copy Markdown

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 8c75389:

Sandbox Source
festive-franklin-f39ug Configuration

@sizebot

sizebot commented Apr 16, 2020

Copy link
Copy Markdown

No significant bundle size changes to report.

Size changes (stable)

Generated by 🚫 dangerJS against 8c75389

@sizebot

sizebot commented Apr 16, 2020

Copy link
Copy Markdown

No significant bundle size changes to report.

Size changes (experimental)

Generated by 🚫 dangerJS against 8c75389

@eps1lon eps1lon closed this Apr 28, 2020
@eps1lon eps1lon deleted the feat/enterkeyhint branch April 28, 2020 19:15
@eps1lon eps1lon restored the feat/enterkeyhint branch April 28, 2020 19:15
@eps1lon eps1lon reopened this Apr 28, 2020
@eps1lon

eps1lon commented Apr 28, 2020

Copy link
Copy Markdown
Collaborator Author

deleted by accident

@stale

stale Bot commented Jul 27, 2020

Copy link
Copy Markdown

This pull request has been automatically marked as stale. If this pull request is still relevant, please leave any comment (for example, "bump"), and we'll keep it open. We are sorry that we haven't been able to prioritize reviewing it yet. Your contribution is very much appreciated.

@stale stale Bot added the Resolution: Stale Automatically closed due to inactivity label Jul 27, 2020
@eps1lon

eps1lon commented Jul 27, 2020

Copy link
Copy Markdown
Collaborator Author

bump

@stale stale Bot removed the Resolution: Stale Automatically closed due to inactivity label Jul 27, 2020
@gaearon gaearon merged commit feb134c into react:master Jul 27, 2020
@gaearon

gaearon commented Jul 27, 2020

Copy link
Copy Markdown
Collaborator

Looks like we forgot to merge. Thanks!

This was referenced Mar 15, 2021
mrizwanashiq pushed a commit to mrizwanashiq/react that referenced this pull request Jun 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants