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

Clarify use case for aria-rowindex on cell/gridcell elements #1853

Closed
scottaohara opened this issue Dec 8, 2022 · 3 comments
Closed

Clarify use case for aria-rowindex on cell/gridcell elements #1853

scottaohara opened this issue Dec 8, 2022 · 3 comments
Assignees

Comments

@scottaohara
Copy link
Member

Presently, aria-rowindex states

Authors MAY also place aria-rowindex on all of the children or owned elements of each row.

but, it doesn't indicate why anyone would do this. The example provided to demonstrate this 'may' specifies the aria-rowindex on the parent row as well as its gridcell children, which doesn't seem particularly necessary or helpful?

So, is there a reason one would do this? It isn't clear from the spec. Even if each cell had the aria-rowindex on it instead of the parent row, I'd question why the effort was taken to apply that attribute on multiple cells rather on the single parent? Are the potentials for conflicts if the row has no specified index, but its children cells do which may not match the current indexing that's actually present in the DOM?

If this attribute does have a good use case for being used on cell / gridcell, then we should update the text to better call out what that is.

If it doesn't, then I'm not sure why this is allowed. But, also wonder if there isnt' something useful that could be done with it?

For instance, recently I was asked if setting aria-rowindex on a cell could mitigate the need for specifying rows all together. So someone could create a series of sibling cells, and then set the necessary rowindex on them to divide the cells into the necessary rows without having to split up the cells into different container elements.

That would be kinda useful, actually, and based on the current spec text, i can absolutely see why they'd think this was possible.

@pkra
Copy link
Member

pkra commented Jan 12, 2023

As per call today: this looks like duplicate of #1138

@scottaohara
Copy link
Member Author

note to self: need to run some tests for current support for aria-rowindex on cells and what gets exposed, if anything.

@pkra
Copy link
Member

pkra commented Jun 28, 2023

Closing this in favor of #1138

@pkra pkra closed this as completed Jun 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants