-
Notifications
You must be signed in to change notification settings - Fork 6.8k
feat(material/form-field-custom-control): custom phone input autoadvance #19650
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
Conversation
Automatically focus the next input when done with the previous one. Allow autofocusing backward when pressing backspace. onContainerClick now focuses on the last filled input on click. Using maxLength on the inputs to not allow more characters than the size of the input. Fixes #13195
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice, thanks for making this demo a little more full featured
...onents-examples/material/form-field/form-field-custom-control/example-tel-input-example.html
Outdated
Show resolved
Hide resolved
...-examples/material/form-field/form-field-custom-control/form-field-custom-control-example.ts
Outdated
Show resolved
Hide resolved
...-examples/material/form-field/form-field-custom-control/form-field-custom-control-example.ts
Outdated
Show resolved
Hide resolved
...-examples/material/form-field/form-field-custom-control/form-field-custom-control-example.ts
Outdated
Show resolved
Hide resolved
...onents-examples/material/form-field/form-field-custom-control/example-tel-input-example.html
Outdated
Show resolved
Hide resolved
…umber input Updates styling for form-field-custom-control-example component, multi-line elements in template due to them getting long, argument spacing. Fixes #13195
@mmalerba thanks for your feedback. I applied the requested changes. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks! - 1 more minor nit then it's good to go
...-examples/material/form-field/form-field-custom-control/form-field-custom-control-example.ts
Outdated
Show resolved
Hide resolved
…nce (angular#19650) * feat(material/form-field-custom-control): custom phone input autoadvance Automatically focus the next input when done with the previous one. Allow autofocusing backward when pressing backspace. onContainerClick now focuses on the last filled input on click. Using maxLength on the inputs to not allow more characters than the size of the input. Fixes angular#13195 * style(material/form-field-custom-control): styling update for phone number input Updates styling for form-field-custom-control-example component, multi-line elements in template due to them getting long, argument spacing. Fixes angular#13195 * style(material/form-field-custom-control): removes unnecessary !! from autoFocusNext * Merge remote-tracking branch 'upstream/master' into phone-input-auto-advance
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
Automatically focus the next input when done with the previous one. Allow autofocusing backward when pressing backspace. onContainerClick now focuses on the last filled input on click. Using maxLength on the inputs to not allow more characters than the size of the input.
https://medium.com/@wildhammer/angular-material-phone-input-2283c7bc3372
Fixes #13195