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.

Sign up for GitHub

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

Jump to bottom

[TextInput] Add a default max length based on input type #986

Closed

Tracked by

billhimmelsbach opened this issue Oct 3, 2024 · 0 comments · Fixed by #988

Closed

Tracked by

[TextInput] Add a default max length based on input type #986

billhimmelsbach opened this issue Oct 3, 2024 · 0 comments · Fixed by #988

Assignees

Contributor

billhimmelsbach commented Oct 3, 2024 •

edited by tanner-ricks

Loading

We should only allow a max of around 255 characters for text inputs, with the exception of email (320). We'll be adding it to the TextInput wrapper component.
add a util function that checks a given input field to see if it passes the check
use that util function to check our existing inputs

billhimmelsbach assigned tanner-ricks

tanner-ricks linked a pull request

that will close this issue

[TextInput] Add a default max length based on input type #988

Merged

billhimmelsbach mentioned this issue

[Story] Text inputs to only accept valid input lengths cfpb/sbl-project#285

Open

2 tasks

tanner-ricks closed this as completed in #988

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment