fix: makes the beforetoggle event listener passive and move to constructor

Author: iOvergaard

packages/uui-popover-container/lib/uui-popover-container.element.ts

@@ -72,14 +72,20 @@ export class UUIPopoverContainerElement extends LitElement {
   #sizeObserver: ResizeObserver | null = null;
   #size: { width: number; height: number } = { width: 0, height: 0 };
 
+  constructor() {
+    super();
+
+    this.addEventListener('beforetoggle', this.#onBeforeToggle, {
+      passive: true,
+    });
+  }
+
   connectedCallback(): void {
+    super.connectedCallback();
     if (!this.hasAttribute('popover')) {
       this.setAttribute('popover', '');
     }
 
-    super.connectedCallback();
-    this.addEventListener('beforetoggle', this.#onBeforeToggle);
-
     if (!this.#sizeObserver) {
       this.#sizeObserver = new ResizeObserver(entries => {
         const element = entries[0]; // should be only one
@@ -101,7 +107,6 @@ export class UUIPopoverContainerElement extends LitElement {
 
   disconnectedCallback(): void {
     super.disconnectedCallback();
-    this.removeEventListener('beforetoggle', this.#onBeforeToggle);
     this.#stopScrollListener();
     this.#sizeObserver?.disconnect();
     this.#sizeObserver = null;