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

<hr>-in-<select> semantics #480

Closed
annevk opened this issue May 2, 2023 · 1 comment · Fixed by #504
Closed

<hr>-in-<select> semantics #480

annevk opened this issue May 2, 2023 · 1 comment · Fixed by #504
Assignees

Comments

@annevk
Copy link
Member

annevk commented May 2, 2023

To match the visual aesthetics, perhaps sibling <hr> elements, when children of a <select> element, should be presented as one to the end user.

You can use the following data: URL for demonstration purposes (works at least in Chromium and WebKit on macOS): data:text/xml,<select%20xmlns="http://www.w3.org/1999/xhtml"><option>1</option><hr/><hr/><option>2</option></select>.

This might become more common due to whatwg/html#9124.

@scottaohara
Copy link
Member

Per the way that AT currently behave with select elements, these HRs likely aren't going to be exposed at all to users, since users shouldn't be able to access them directly.

May be worth at least a comment in the mapping table to state as much, or that in this context they should be treated as presentational elements.

that said, this could be very different for their potential use in the proposed selectmenu - if they were to behave similarly there.
Or in my proposal to allow hr to separate li elements within a list container (ul, ol, menu)... there they'd definitely need to be exposed as one would not necessarily be only navigating by focusable elements in that case.

scottaohara added a commit that referenced this issue Oct 3, 2023
closes #480 

Outside of browsers / AT significantly changing how people interact with the UA-provided listboxs, I don't see how the use of hr can be more than declarative decoration, since the element will not be directly accessible to anyone using AT.

If anyone thinks otherwise, and browsers/AT are willing to make changes to how this component has been behaving/exposed, we can discuss and change the PR.
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 a pull request may close this issue.

2 participants