[UPD] add ol.control.Button disable #1081

Viglino · Jun 27, 2024 · 5b5fe0c · 5b5fe0c
1 parent 1e98bb2
commit 5b5fe0c
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 7 deletions.
diff --git a/src/control/Button.js b/src/control/Button.js
@@ -74,6 +74,23 @@ var ol_control_Button = class olcontrolButton extends ol_control_Control {
     else
       ol_ext_element.hide(this.element);
   }
+  /**
+   * Test if the control is disabled.
+   * @return {bool}
+   * @api stable
+   */
+  getDisable() {
+    var button = this.element.querySelector('button');
+    return button && button.disabled;
+  }
+  /** Disable the control button. 
+   * @param {bool} b disable (or enable) the control, default false (enable)
+   * @api stable
+   */
+  setDisable(b) {
+    if (this.getDisable() == b) return;
+    this.element.querySelector('button').disabled = b;
+  }
   /**
    * Set the button title
    * @param {string} title

diff --git a/src/control/Toggle.js b/src/control/Toggle.js
@@ -103,22 +103,20 @@ var ol_control_Toggle = class olcontrolToggle extends ol_control_Button {
   }
   /**
    * Test if the control is disabled.
-   * @return {bool}.
+   * @return {bool}
    * @api stable
    */
   getDisable() {
-    var button = this.element.querySelector("button");
+    var button = this.element.querySelector('button');
     return button && button.disabled;
   }
   /** Disable the control. If disable, the control will be deactivated too.
   * @param {bool} b disable (or enable) the control, default false (enable)
   */
   setDisable(b) {
-    if (this.getDisable() == b)
-      return;
-    this.element.querySelector("button").disabled = b;
-    if (b && this.getActive())
-      this.setActive(false);
+    if (this.getDisable() == b) return;
+    this.element.querySelector('button').disabled = b;
+    if (b && this.getActive()) this.setActive(false);
 
     this.dispatchEvent({ type: 'change:disable', key: 'disable', oldValue: !b, disable: b });
   }