fix: make setupTooltip a private method

muxinc · Sep 26, 2024 · aea5b42 · aea5b42
1 parent dbdec7e
commit aea5b42
Showing 1 changed file with 13 additions and 13 deletions.
diff --git a/src/js/media-chrome-button.ts b/src/js/media-chrome-button.ts
@@ -285,7 +285,19 @@ class MediaChromeButton extends globalThis.HTMLElement {
 
     globalThis.customElements
       .whenDefined('media-tooltip')
-      .then(this.setupTooltip.bind(this));
+      .then(this.#setupTooltip.bind(this));
+  }
+
+  // Called when we know the tooltip is ready / defined
+  #setupTooltip() {
+    this.addEventListener('mouseenter', this.#positionTooltip);
+    this.addEventListener('focus', this.#positionTooltip);
+    this.addEventListener('click', this.#clickListener);
+
+    const initialPlacement = this.tooltipPlacement;
+    if (initialPlacement && this.tooltipEl) {
+      this.tooltipEl.placement = initialPlacement;
+    }
   }
 
   disconnectedCallback() {
@@ -319,18 +331,6 @@ class MediaChromeButton extends globalThis.HTMLElement {
    * @argument {Event} e
    */
   handleClick(e) {} // eslint-disable-line
-
-  // Called when we know the tooltip is ready / defined
-  setupTooltip() {
-    this.addEventListener('mouseenter', this.#positionTooltip);
-    this.addEventListener('focus', this.#positionTooltip);
-    this.addEventListener('click', this.#clickListener);
-
-    const initialPlacement = this.tooltipPlacement;
-    if (initialPlacement && this.tooltipEl) {
-      this.tooltipEl.placement = initialPlacement;
-    }
-  }
 }
 
 if (!globalThis.customElements.get('media-chrome-button')) {