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

fix: added error param, test, story #2488

Merged
merged 5 commits into from
Jul 11, 2023
Merged

fix: added error param, test, story #2488

merged 5 commits into from
Jul 11, 2023

Conversation

werdnanoslen
Copy link
Contributor

Summary

Adds property to FileInput component to allow error text inside component to be customized. Before, it was hardcoded making it inappropriate for mobile and non-English contexts.

Related Issues or PRs

Resolves #2456

How To Test

  • Give custom text to errorText
  • Ensure all other file input functions behave as before
  • Or run storybook, go here and try drag-dropping anything to the field

Screenshots (optional)

Screenshot 2023-07-10 at 2 42 09 PM

@werdnanoslen werdnanoslen changed the title added error param, test, story fix: added error param, test, story Jul 10, 2023
@werdnanoslen werdnanoslen merged commit 42da85d into main Jul 11, 2023
@werdnanoslen werdnanoslen deleted the an-inputerror-2456 branch July 11, 2023 02:09
@haworku
Copy link
Contributor

haworku commented Jul 11, 2023

Thanks!

gidjin pushed a commit that referenced this pull request Jul 14, 2023
## [5.1.0](5.0.0...5.1.0) (2023-07-14)


### Features

* added customizable text control to file input ([#2417](#2417)) ([0a387be](0a387be))
* adds unbounded functionality to pagination ([#2418](#2418)) ([f9e7957](f9e7957))


### Bug Fixes

* a11y: adds css to hide search clear button that causes accessibility issues ([#2467](#2467)) ([d19527f](d19527f))
* added error param, test, story ([#2488](#2488)) ([42da85d](42da85d))

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[fix] Parameterize error text on file input
3 participants