Skip to content

Commit

Permalink
Improve docs and inline comment.
Browse files Browse the repository at this point in the history
  • Loading branch information
afercia committed Feb 12, 2025
1 parent 410182d commit 0f9d798
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 1 deletion.
2 changes: 2 additions & 0 deletions packages/components/src/tree-grid/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,8 @@ A tree grid is a hierarchical 2 dimensional UI component, for example it could b

A tree grid allows the user to navigate using arrow keys. Up/down to navigate vertically across rows, and left/right to navigate horizontally between focusables in a row.

To make the keyboard navigation and roving tabindex behaviors work as expected it is important to avoid programmatically setting focus on any of the focusable items in the tree grid. In fact, `RovingTabIndexItem` handles the logic to make only one item navigable with the Tab key at a time. The other items can be navigated with the arrow keys. Triggering a focus event may conflict with the `RovingTabIndexItem` internal logic.

For more information on a tree grid, see the following links:

- https://www.w3.org/TR/wai-aria-practices/examples/treegrid/treegrid-1.html
Expand Down
2 changes: 1 addition & 1 deletion packages/editor/src/components/list-view-sidebar/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,7 @@ export default function ListViewSidebar() {

// This ref refers to the sidebar as a whole.
const sidebarRef = useRef();
// This ref refers to the tab panel.
// This ref refers to the tablist.
const tabsRef = useRef();
// This ref refers to the list view application area.
const listViewRef = useRef();
Expand Down

0 comments on commit 0f9d798

Please sign in to comment.