chore(react-tree-grid): updates LiveMeetingsList story for better navigation #263
Conversation
| {props.attachments && ( | ||
| <TreeGridCell className={styles.attachments}> | ||
| {props.attachments} | ||
| </TreeGridCell> | ||
| )} |
There was a problem hiding this comment.
what is the conclusion on assymetric rows for treegrids? should we have them or not?
There was a problem hiding this comment.
There's no conclusion, we're focusing at the moment on the proper narration when you navigate on a nested level (from row to cells). At the moment there's a long repetition of unreachable content that is narrated once on the row level and again once you go into the cells, we're trying to mitigate that at the moment.
I believe the asymmetry might be an issue when navigating between rows, on a vertical level, we still need to verify that with more screen readers.
There was a problem hiding this comment.
Good point I think would be better to render just empty gridcell.
Because when trying to navigation with NVDA in column navigation ctrl+alt+up/down arrows, then when there is asymmetry, then screen reader jump to different column automatically. What sounds to me confusing, because I want to keep column navigation...
This is good point for next discussion how to solve it properly :)
Anyway if we decide to have it symmetric still we need to figure if we should add aria-label "empty" for empty gridcell or let it handle by screen reader.
Updates
LiveMeetingsListstories to ensure a better narration.