Skip to content

Commit b60cf85

Browse files
committed
Erin feedback
1 parent bcec348 commit b60cf85

File tree

1 file changed

+8
-6
lines changed
  • packages/documentation-site/patternfly-docs/content/AI/Compass

1 file changed

+8
-6
lines changed

packages/documentation-site/patternfly-docs/content/AI/Compass/ReactFlow.md

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -52,10 +52,12 @@ The React Flow package `@xyflow/react` is compatible with PatternFly through cus
5252

5353
### Accessibility considerations
5454

55-
While React Flow is built with accessibility in mind, you should always check that your implementation when paired with PatternFly is accessible via mouse, keyboard, and other assistive technologies like screen readers. Some things to keep in mind include:
55+
While React Flow is built with accessibility in mind, you should always check that your implementation (when paired with PatternFly) is accessible via mouse, keyboard, and other assistive technologies like screen readers.
5656

57-
- Providing an accessible name to the `<ReactFlow>` component, if its default role of "application" is kept or if it is given another semantic role that requires an accessible name.
58-
- Due to the nature of React Flow and howw you are able to click and drag nodes, if your draggable nodes contain other actions inside of them (kebab toggles, links, other action buttons), you should ensure those actions are large enough to prevent misclicks.
59-
- [WCAG 2.5.8 - Target size (Minimum)](https://www.w3.org/WAI/WCAG22/Understanding/target-size-minimum) requires at least 24z24 CSS pixels. While they also state exceptions, because a React Flow node itself can be clicked for some action, you should aim for sticking to this minimum, especially if implementing an action where clicking a node quickly can open a details drawer.
60-
- In general ensure that any actions that can be taken via mouse only can be taken via keyboard as well, whether that be actions inside a node or being able to drag a node around to reposition it.
61-
- Dragging nodes via keyboard should be built into React Flow itself out of the box, but always double check in case.
57+
Additionally, you should align with the following practices:
58+
59+
- Provide an accessible name to the `<ReactFlow>` component, if its default role of "application" is kept or if it is given another semantic role that requires an accessible name.
60+
- Because React Flow supports node clicking and dragging, ensure that actions within draggable nodes are large enough to prevent misclicks (for example, kebab toggles, links, or other action buttons).
61+
- [WCAG 2.5.8: Target size (Minimum)](https://www.w3.org/WAI/WCAG22/Understanding/target-size-minimum) requires targets to be at least 24 by 24 CSS pixels. While there are exceptions, you should still aim for this minimum because a React Flow node itself can be clicked for some actions. This is especially important for an action like opening a details drawer upon a quick click to a node.
62+
- Ensure that any actions that can be taken via mouse only can be taken via keyboard as well, including all actions within a node and node repositioning via dragging.
63+
- Dragging nodes via keyboard should be built into React Flow itself by default, but always double check to confirm.

0 commit comments

Comments
 (0)