question-mark
Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

grid vs. treegrid allowed row attributes

See original GitHub issue

Expectation: aria-posinset, aria-setsize, aria-expanded, and aria-level should all be allowed on role="row", but only when descending from a treegrid.

Actual: aria-posinset and aria-setsize always cause an allowed attribute error on row, while aria-expanded and aria-level never do.

You can see the false positives on the aria-practices example on this page: https://w3c.github.io/aria-practices/examples/treegrid/treegrid-1.html. If you change the parent role to grid then run the test again, you can see that aria-expanded and aria-level are not flagged.

Motivation: Valid treegrid attributes are getting errors flagged, resulting in a stream of bad bugs, and some easily-caught errors like aria-expanded or aria-level on grids and tables are not flagged.


axe-core version: 4.1.1

Issue Analytics

  • State:closed
  • Created 3 years ago
  • Reactions:1
  • Comments:12 (7 by maintainers)

github_iconTop GitHub Comments

1reaction
strakercommented, Jun 2, 2021

We recently merged a PR that allows aria-posinset and aria-setsize to be allowed on row elements to remove the false positive. We’ll work on another PR which will scope the attributes to just treegrid and warn the user if they use it on grid or table.

1reaction
jessiehuffcommented, Mar 30, 2021

I completely agree with @smhigley here. We’re trying to build out this structure in Red Hat’s design system, PatternFly, and we’ve been running into this issue. We currently have axe-core running against our CI/CD build so the PR we had that follows the previous example shared, was being flagged as failing. (This is a preview of our implementation.) We set it to ignore that specific example since it seems to be the best structure we tested, but we worry about the other products who might use this variation. We’ve been encouraging Red Hat products to also check their build against axe-core so we’re concerned that they might see these failures and be confused. I’d love to see these attributes supported!

Read more comments on GitHub >

github_iconTop Results From Across the Web

XML - Rows - TreeGrid
TreeGrid can contain variable data rows, fixed data rows, fixed special rows, solid space rows and solid special rows. According to type are...
Read more >
ARIA: treegrid role - Accessibility - MDN Web Docs - Mozilla
The treegrid role identifies an element as being grid whose rows can be expanded and collapsed in the same manner as for a...
Read more >
Treegrid | APG | WAI - W3C
In a treegrid both rows and cells are focusable. Every row and cell contains a focusable element or is itself focusable, regardless of...
Read more >
lightning-tree-grid - documentation - Salesforce Developers
The first column specifies the utility:event icon for all rows using the iconName cell attribute, and the icon displays to the left of...
Read more >
JavaScript TreeGrid - Configuration | DHTMLX Suite 7 Docs
You will find the full list of the configuration properties of a TreeGrid column ... "source" - a grid allows dragging its rows...
Read more >

github_iconTop Related Medium Post

No results found

github_iconTop Related StackOverflow Question

No results found

github_iconTroubleshoot Live Code

Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free

github_iconTop Related Reddit Thread

No results found

github_iconTop Related Hackernoon Post

No results found

github_iconTop Related Tweet

No results found

github_iconTop Related Dev.to Post

No results found

github_iconTop Related Hashnode Post

No results found