From d7083c5303775450ff27063857e61eb5b653e418 Mon Sep 17 00:00:00 2001 From: Eemeli Aro Date: Fri, 2 Apr 2021 11:34:45 +0300 Subject: [PATCH] [added] docs note about setAppElement not pruning removed nodes --- docs/accessibility/index.md | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/docs/accessibility/index.md b/docs/accessibility/index.md index 505ae9f8..ca96b763 100644 --- a/docs/accessibility/index.md +++ b/docs/accessibility/index.md @@ -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