From 6799b87cc6bc9c456b65e37d72f3003f45c47737 Mon Sep 17 00:00:00 2001 From: toasted-nutbread Date: Sat, 10 Oct 2020 10:47:02 -0400 Subject: Make animation optional for Modal.setVisible (#903) --- ext/bg/js/settings/modal.js | 20 +++++++++++--------- 1 file changed, 11 insertions(+), 9 deletions(-) (limited to 'ext/bg') diff --git a/ext/bg/js/settings/modal.js b/ext/bg/js/settings/modal.js index 0cba167d..0b09f6ca 100644 --- a/ext/bg/js/settings/modal.js +++ b/ext/bg/js/settings/modal.js @@ -41,7 +41,7 @@ class Modal extends EventDispatcher { } } - setVisible(value) { + setVisible(value, animate=true) { value = !!value; if (this._useJqueryModal()) { this._getWrappedNode().modal(value ? 'show' : 'hide'); @@ -55,19 +55,21 @@ class Modal extends EventDispatcher { } if (value) { - classList.add(this._openingClassName); + if (animate) { classList.add(this._openingClassName); } + classList.remove(this._closingClassName); getComputedStyle(this._node).getPropertyValue('display'); // Force update of CSS display property, allowing animation classList.add(this._visibleClassName); - classList.remove(this._closingClassName); - classList.remove(this._openingClassName); + if (animate) { classList.remove(this._openingClassName); } this._node.focus(); } else { - classList.add(this._closingClassName); + if (animate) { classList.add(this._closingClassName); } classList.remove(this._visibleClassName); - this._closeTimer = setTimeout(() => { - this._closeTimer = null; - classList.remove(this._closingClassName); - }, this._closingAnimationDuration); + if (animate) { + this._closeTimer = setTimeout(() => { + this._closeTimer = null; + classList.remove(this._closingClassName); + }, this._closingAnimationDuration); + } } } } -- cgit v1.2.3