diff options
author | Andrew Luca <thendrluca@gmail.com> | 2018-10-19 01:26:11 +0300 |
---|---|---|
committer | Johann-S <johann.servoire@gmail.com> | 2018-10-30 18:28:32 +0300 |
commit | bd2851938a5a5764adac008613825fb2daca0326 (patch) | |
tree | b195225bf12dade85acb7be71ad7513a0ef7d4ad /js/src | |
parent | 65dc8c907048111d7895b64da1207023ff4c9992 (diff) |
test(Modal): check if modal is disposed
Diffstat (limited to 'js/src')
-rw-r--r-- | js/src/modal.js | 12 |
1 files changed, 10 insertions, 2 deletions
diff --git a/js/src/modal.js b/js/src/modal.js index 704b022472..0004fe8bbe 100644 --- a/js/src/modal.js +++ b/js/src/modal.js @@ -196,9 +196,17 @@ class Modal { } dispose() { - $.removeData(this._element, DATA_KEY) + [window, this._element, this._dialog] + .forEach((htmlElement) => $(htmlElement).off(EVENT_KEY)) + + /** + * `document` has 2 events `Event.FOCUSIN` and `Event.CLICK_DATA_API` + * Do not move `document` in `htmlElements` array + * It will remove `Event.CLICK_DATA_API` event that should remain + */ + $(document).off(Event.FOCUSIN) - $(window, document, this._element, this._backdrop).off(EVENT_KEY) + $.removeData(this._element, DATA_KEY) this._config = null this._element = null |