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

feat: porting of volto-form-block autocomplete changes #779

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

sabrina-bongiovanni
Copy link
Collaborator

To be merged after collective/volto-form-block#118 is merged and released on io-Comune.

Porting of changes to use autocomplete field on form fields.

Accessibility tools point out as errors the fact that form fields do not have an autocomplete attribute.
This PR aims to allow editors to set autocomplete values according to the form field role.

Since semantic-ui does not support autocomplete as a prop, I added the react-aria-components library as a dependency from this addon. However, since this library is going to be applied to volto as a whole, someday probably it won't be necessary to have it installed in this addon.
I used this library as it is the one that will be implemented, but of course I'm open to discuss and making changes in case it is not the best option.

@sabrina-bongiovanni sabrina-bongiovanni self-assigned this Sep 30, 2024
@sabrina-bongiovanni sabrina-bongiovanni added the a11y-accessibility Fixes accessibility issues or adds accessibility features label Sep 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a11y-accessibility Fixes accessibility issues or adds accessibility features
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant