Skip to content

Commit

Permalink
[added] docs note about setAppElement not pruning removed nodes
Browse files Browse the repository at this point in the history
eemeli authored and diasbruno committed Apr 2, 2021

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
1 parent e1807ce commit d7083c5
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion docs/accessibility/index.md
Original file line number Diff line number Diff line change
@@ -24,7 +24,10 @@ Modal.setAppElement(document.getElementById('root'));
```

Using a selector that matches multiple elements or passing a list of DOM
elements will hide all of the elements.
elements will hide all of the elements. Note that this list won't be
automatically pruned if elements are removed from the DOM, so you may want to
call `Modal.setAppElement` when any such changes are made, or pass a live
HTMLCollection as the value.

If you are already applying the `aria-hidden` attribute to your app content
through other means, you can pass the `ariaHideApp={false}` prop to your modal

0 comments on commit d7083c5

Please sign in to comment.