-
Notifications
You must be signed in to change notification settings - Fork 77
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
Focus State Behavior for v3 List Items Appears Unexpected #11078
Comments
@geospatialem I think this focus behavior should be categorized as a bug and I think it's been with us for while. I believe this is the focus state in question where only the content area is focused: Ideally, focus should only be on the entire item or interactive elements, such as the drag handle, close button, or slotted actions. |
Confirmed with @ashetland this has been occurring for a while, at least since 2.6 and in our latest 2.13, but this is something we'd like to address ASAP, ideally for 3.0, but if not, shortly thereafter. |
This should follow the treegrid role guidelines. https://www.w3.org/WAI/ARIA/apg/patterns/treegrid/examples/treegrid-1/#exampleusageoptions For example, our options are: Rows are focused first, but cells can be focused: |
I think the treegrid role requires cells to be focusable.
|
Thanks for the insight, @driskull. While the W3C guidelines are valuable for true table structures, these components lack distinct columns or cells, so applying table-based focus behavior feels misaligned. Focus states here should prioritize the component’s interactive elements rather than mimic treegrid conventions. Thoughts? |
@AdamWMoqrane this component is using and following the treegrid role for accessibility purposes. It does have cells for actions and content. Interactivity needs to be based on proper roles for accessibility compliance. As far as I'm aware, there isn't a better role here. |
I may be reading the spec wrong, but in the "rows first" scenario it implies to me that cells can be focused, but not that they have to be. Is it be possible to exclude some cells from focus and not wreak havoc with AT? |
It seems like all cells need to be focusable. We can adjust what a cell is if need be. What I think we could do that would make this less noticable is to only focus on the row when a cell is clicked and not a cell. A user would need to use the keyboard arrows to get to a specific cell. That seems to be what they are doing in the demo. |
Seems like an approach we could test out. Thanks for digging into this @driskull! |
**Related Issue:** #11078 ## Summary - no longer focuses on cells when a cell is focused via click - only a row or cell can be focusable via tabindex at a time - update tests
Installed and assigned for verification. |
Description
The focus states for v3 List components are inconsistent, leading to alignment issues and unexpected focus behavior. This problem seems related to the structure of the component and the keyboard navigation of each "table cell" within the List. This issue persists even when using the
icon-start
property introduced in 3.0.User Stories
As a product designer working with List components, I want the focus states to be standardized and consistent so that users have a predictable and accessible experience when navigating through list items.
Acceptance Criteria
content-start
oricon-start
slots does not introduce misaligned or unintended focus behavior.Relevant Info
icon-start
with v3 List components.Helpful Details
content-start
slots, which exacerbates the misalignment issues.icon-start
slot improves alignment slightly but does not completely resolve focus behavior.Priority impact
impact - p1 - need for current milestone
Esri team
ArcGIS Knowledge
The text was updated successfully, but these errors were encountered: