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(file-uploader): ensure form requirements are visible #6867

Conversation

emyarod
Copy link
Member

@emyarod emyarod commented Sep 21, 2020

Closes #6866

This PR adds a style rule which ensures that form requirements for invalid file uploader contents are visible

Testing / Reviewing

Add an invalid file in a file uploader demo and confirm that the error details are visible

@netlify
Copy link

netlify bot commented Sep 21, 2020

Deploy preview for carbon-elements ready!

Built with commit 083cf48

https://deploy-preview-6867--carbon-elements.netlify.app

@netlify
Copy link

netlify bot commented Sep 21, 2020

Deploy preview for carbon-components-react ready!

Built without sensitive environment variables with commit 083cf48

https://deploy-preview-6867--carbon-components-react.netlify.app

@joshblack
Copy link
Contributor

bump @andreancardona when you get a sec today

Copy link
Contributor

@andreancardona andreancardona left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@emyarod looks good!

@tw15egan tw15egan requested review from a team and johnbister and removed request for a team September 22, 2020 21:31
@johnbister
Copy link
Contributor

Looks good to me!
Screen Shot 2020-09-22 at 6 50 10 PM

@kodiakhq kodiakhq bot merged commit 511ffcf into carbon-design-system:master Sep 23, 2020
@emyarod emyarod deleted the 6866-file-uploader-form-requirements-visibility branch September 23, 2020 15:11
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.

File uploader form requirements are hidden
5 participants