@lion/ui 0.14.3 → 0.15.1

package/components/overlays/src/singleton.js

@@ -1,6 +1,16 @@
-import { singletonManager } from 'singleton-manager';
+import { singletonManager, lazifyInstantiation } from 'singleton-manager';
 import { OverlaysManager } from './OverlaysManager.js';
 
-export const overlays =
-  /** @type {OverlaysManager} */
-  (singletonManager.get('@lion/ui::overlays::0.x')) || new OverlaysManager();
+/**
+ * @returns {OverlaysManager}
+ */
+function getOverlaysManager() {
+  if (!singletonManager.has('@lion/ui::overlays::0.x')) {
+    const overlaysManager = new OverlaysManager();
+    singletonManager.set('@lion/ui::overlays::0.x', overlaysManager);
+  }
+
+  return singletonManager.get('@lion/ui::overlays::0.x');
+}
+
+export const overlays = lazifyInstantiation(getOverlaysManager);

package/components/overlays/test/OverlayController.test.js

@@ -799,6 +799,9 @@ describe('OverlayController', () => {
           const { parentOverlay, childOverlay } = await createNestedEscControllers(parentContent);
           await mimicEscapePress(childOverlay.contentNode);
 
+          // without this line, the test is unstable on FF sometimes
+          await aTimeout(0);
+
           expect(parentOverlay.isShown).to.be.false;
           expect(childOverlay.isShown).to.be.true;
 
@@ -1040,7 +1043,7 @@ describe('OverlayController', () => {
         // Don't hide on inside shadowDom click
         /** @type {ShadowRoot} */
         // @ts-expect-error
-        (ctrl.contentNode.querySelector(tagString).shadowRoot).querySelector('button').click();
+        ctrl.contentNode.querySelector(tagString).shadowRoot.querySelector('button').click();
 
         await aTimeout(0);
         expect(ctrl.isShown).to.be.true;