diff options
| author | Ryan Weaver <[email protected]> | 2021-05-31 05:35:59 -0400 |
|---|---|---|
| committer | GitHub <[email protected]> | 2021-05-31 12:35:59 +0300 |
| commit | 0cb70e214ffd24070fa9e312746953faa0b8a305 (patch) | |
| tree | 0bc514423e0c217b3dd01ee0cbbb7b052b722c52 /js/tests | |
| parent | e818c7cd5ddafdf68f627e5e3547bdd281bc5a5b (diff) | |
| download | bootstrap-0cb70e214ffd24070fa9e312746953faa0b8a305.tar.xz bootstrap-0cb70e214ffd24070fa9e312746953faa0b8a305.zip | |
Changing Backdrop rootElement to default to a string (#34092)
The current config can cause the "body" to become stale. Specifically,
if the entire body element is swapped out for a new body element, then
the backdrop will continue to append itself to the original body element,
since it's stored in memory as a reference on this object.
This also no longer allows an explicit null to be passed to Backdrop's rootElement
This still accomplishes the laziness of "not finding the rootElement
until the Backdrop is created" to avoid problems of the JavaScript
being included inside <head> (so, before body is available).
Diffstat (limited to 'js/tests')
| -rw-r--r-- | js/tests/unit/util/backdrop.spec.js | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/js/tests/unit/util/backdrop.spec.js b/js/tests/unit/util/backdrop.spec.js index 195d5067c..3150ba14d 100644 --- a/js/tests/unit/util/backdrop.spec.js +++ b/js/tests/unit/util/backdrop.spec.js @@ -243,10 +243,10 @@ describe('Backdrop', () => { }) }) - it('Should default parent element to "document.body" when config value is null', done => { + it('Should find the rootElement if passed as a string', done => { const instance = new Backdrop({ isVisible: true, - rootElement: null + rootElement: 'body' }) const getElement = () => document.querySelector(CLASS_BACKDROP) instance.show(() => { |