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

No Label for Search Block Dropdown Form element #5400

Open
ThomasKindermann opened this issue Nov 14, 2023 · 2 comments
Open

No Label for Search Block Dropdown Form element #5400

ThomasKindermann opened this issue Nov 14, 2023 · 2 comments
Labels

Comments

@ThomasKindermann
Copy link
Contributor

Describe the bug
The "sort by" dropdown in the Search block doesn't have a label.
It should have a label for A11y.

To Reproduce
Steps to reproduce the behavior:

  1. Go to any volto page
  2. Create a Page with the Search block.
  3. Use an A11y Tool or inspect the Dropdown
  4. Dropdown / Input Form element does not have a label.

Expected behavior
Any Form Elements should have a label for screen readers.

Screenshots
Screenshot 2023-11-14 at 12 01 05

  • OS: OSX
  • Browser Chrome
  • Volto 17.0.0-alpha.24
  • Plone 6.0.6
  • plone.restapi 8.41.0
@pranayjoshi
Copy link

pranayjoshi commented Nov 24, 2023

Hi @ThomasKindermann , I am working on this. Is 'Sort by Dropdown' fine as a label? or shall I use the dropdown value as the label?

pranayjoshi added a commit to pranayjoshi/volto that referenced this issue Nov 25, 2023
pranayjoshi added a commit to pranayjoshi/volto that referenced this issue Nov 25, 2023
pranayjoshi added a commit to pranayjoshi/volto that referenced this issue Nov 26, 2023
@stevepiercy
Copy link
Collaborator

In English the phrase preceding the sort input should be Sort by, not Sort on. I hate to pile on to an issue, but since it is part of the same block, it may as well be corrected at the same time. I do not suggest refactoring the code to replace sortOn* names, as I don't know what impact that would have and going that far might be a breaking change.

For the sort input, I would suggest the label Sort by selected option. I thought about just selected option, but I doubt that screen readers would be able to connect the text before the input to the actual input.

The label for the sort input must be internationalized. See https://6.docs.plone.org/volto/recipes/i18n.html.

pranayjoshi added a commit to pranayjoshi/volto that referenced this issue Nov 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants