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

Allow posinset/setsize for toolbars #2158

Open
scottaohara opened this issue Apr 15, 2024 · 1 comment · May be fixed by #2170
Open

Allow posinset/setsize for toolbars #2158

scottaohara opened this issue Apr 15, 2024 · 1 comment · May be fixed by #2170
Assignees

Comments

@scottaohara
Copy link
Member

This is a follow on issue to #1721, specifically this comment/discussion in the minutes about allowing toolbars to either implicitly add an X of Y to controls within - or to allow authors to add an X of Y to controls within a toolbar via aria-posinset and aria-setsize.

scottaohara added a commit that referenced this issue Apr 15, 2024
This PR attempts to close #1721

Per the discussion we had around the issue, the potential usability and wcag issues this could cause, there were few pros to relaxing how one could create a grouping of radio buttons.  The issue of how would one name such a group if there is no actual "group" container to name was a very good point.  Though HTML allows for such loose grouping of their radio buttons, they do not have a solution for all the potential a11y problems it can solve.

So, this PR takes a different approach to resolving the issue, and instead adds a note to clarify that a radiogroup does not have to be limited to just radio buttons, while ALSO pointing out the potential UX issues of adding other arbitrary/focusable elements.

Maybe something could even be added to overtly state that "hey if you want to group radio buttons like you can in HTML, then use HTML radio buttons!"

Happy to wordsmith this further (examples might be good?)  Or entertain other solutions (we could close the issue as won't fix, but i recall being told we should do _something_ rather than just that).

I created #2158 as a separate issue to resolve, since a lot of the conversation about this concept touched on the topic now logged in that issue.  and I didn't want resolving this issue/pr to lose that separate though related topic.
@spectranaut
Copy link
Contributor

Discussed briefly in today's triage: https://www.w3.org/2024/04/18-aria-minutes.html#t01

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants