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

Backport 'Fix dynamic upload file field required indicator + make option naming consistent' to v0.27 #10541

Merged

Conversation

alecslupu
Copy link
Contributor

🎩 What? Why?

Backport #10497 to v0.27

♥️ Thank you!

… consistent (#10497)

* Fix file field required indicator + consistency on option naming

- Fix missing required file indicator from the dynamic upload file
  field label
- Refactor the `optional` option for the file field and upload
  modal to `required` (inverse option) to maintain concistency
  over the option naming for all form fields

* Fix a11y issue with the label `for` attribute in the upload label
@alecslupu alecslupu force-pushed the backport/0.27/fix-dynamic-upload-file-field--10497 branch from 6c35c3e to b46eb25 Compare March 13, 2023 14:41
@alecslupu alecslupu requested a review from a team March 13, 2023 19:09
@alecslupu alecslupu added this to Pending review from Product in Maintainers via automation Mar 13, 2023
@fblupi fblupi merged commit 9085556 into release/0.27-stable Mar 14, 2023
4 checks passed
Maintainers automation moved this from Pending review from Product to Done Mar 14, 2023
@fblupi fblupi deleted the backport/0.27/fix-dynamic-upload-file-field--10497 branch March 14, 2023 13:30
@alecslupu alecslupu added this to the 0.27.3 milestone Jul 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging this pull request may close these issues.

None yet

3 participants