Skip to content

Commit d7083c5

Browse files
eemelidiasbruno
authored andcommitted
[added] docs note about setAppElement not pruning removed nodes
1 parent e1807ce commit d7083c5

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

docs/accessibility/index.md

+4-1
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,10 @@ Modal.setAppElement(document.getElementById('root'));
2424
```
2525

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

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

0 commit comments

Comments
 (0)