@lion/ui 0.14.1 → 0.14.2

package/components/dialog/test/lion-dialog.test.js

@@ -1,9 +1,20 @@
 /* eslint-disable lit-a11y/no-autofocus */
-import { expect, fixture as _fixture, html, unsafeStatic, aTimeout } from '@open-wc/testing';
+import {
+  expect,
+  fixture as _fixture,
+  html,
+  unsafeStatic,
+  aTimeout,
+  defineCE,
+  waitUntil,
+} from '@open-wc/testing';
+import { cache } from 'lit/directives/cache.js';
+import { LitElement, nothing } from 'lit';
 import { runOverlayMixinSuite } from '../../overlays/test-suites/OverlayMixin.suite.js';
 import { isActiveElement } from '../../core/test-helpers/isActiveElement.js';
 import '../test-helpers/test-router.js';
 import '@lion/ui/define/lion-dialog.js';
+import '@lion/ui/define/lion-tabs.js';
 
 /**
  * @typedef {import('../src/LionDialog.js').LionDialog} LionDialog
@@ -296,5 +307,107 @@ describe('lion-dialog', () => {
       await dialogAfterRouteChange._overlayCtrl._showComplete;
       expect(dialogAfterRouteChange.opened).to.be.true;
     });
+
+    it('should close the popup dialog after rendered from cache', async () => {
+      /**
+       *
+       * @param {Event} e
+       * @returns
+       */
+      const closeButtonHandler = e =>
+        e.target?.dispatchEvent(new Event('close-overlay', { bubbles: true }));
+      const dialog = html` <lion-dialog>
+        <button slot="invoker" class="invoker-button">Click me to open dialog</button>
+        <div slot="content" class="demo-dialog-content">
+          Hello! You can close this dialog here:
+          <button class="close-button" @click="${closeButtonHandler}">⨯</button>
+        </div>
+      </lion-dialog>`;
+
+      /**
+       * Note, inactive tab content is **destroyed** on every tab switch.
+       */
+      class Wrapper extends LitElement {
+        static properties = {
+          ...super.properties,
+          activeTabIndex: { type: Number },
+        };
+
+        constructor() {
+          super();
+          this.activeTabIndex = 0;
+        }
+
+        /**
+         * @param {number} index
+         */
+        changeActiveTabIndex(index) {
+          this.activeTabIndex = index;
+        }
+
+        render() {
+          const changeActiveTabIndexRef = this.changeActiveTabIndex.bind(this);
+          return html`
+            <lion-tabs>
+              <button slot="tab" class="first-button" @click=${() => changeActiveTabIndexRef(0)}>
+                First
+              </button>
+              <p slot="panel">${cache(this.activeTabIndex === 0 ? dialog : nothing)}</p>
+              <button slot="tab" class="second-button" @click=${() => changeActiveTabIndexRef(1)}>
+                Second
+              </button>
+              <p slot="panel">Info page with lots of information about us.</p>
+            </lion-tabs>
+          `;
+        }
+      }
+
+      const wrapperFixture = /** @type {(arg: TemplateResult) => Promise<Wrapper>} */ (_fixture);
+      const tagString = defineCE(Wrapper);
+      const wrapperTag = unsafeStatic(tagString);
+      const wrapperElement = /** @type {Wrapper} */ (
+        await wrapperFixture(html`<${wrapperTag}></${wrapperTag}>`)
+      );
+      await wrapperElement.updateComplete;
+      const wrapperElementShadowRoot = wrapperElement.shadowRoot;
+      /**
+       * @returns { HTMLElement | null | undefined }
+       */
+      const getFirstButton = () => wrapperElementShadowRoot?.querySelector('.first-button');
+      /**
+       * @returns { HTMLElement | null | undefined }
+       */
+      const getSecondButton = () => wrapperElementShadowRoot?.querySelector('.second-button');
+      /**
+       * @returns { HTMLElement | null | undefined }
+       */
+      const getInvokerButton = () => wrapperElementShadowRoot?.querySelector('.invoker-button');
+      /**
+       * @returns { HTMLElement | null | undefined }
+       */
+      const getCloseButton = () => wrapperElementShadowRoot?.querySelector('.close-button');
+      /**
+       * @returns { Element | null | undefined }
+       */
+      const getDialog = () =>
+        wrapperElementShadowRoot?.querySelector('lion-dialog')?.shadowRoot?.querySelector('dialog');
+      // @ts-ignore
+      const isDialogVisible = () => getDialog()?.checkVisibility() === true;
+      const isDialogRendered = () =>
+        !!wrapperElement.shadowRoot?.querySelector('lion-dialog')?.shadowRoot?.childNodes.length;
+      getInvokerButton()?.click();
+      await waitUntil(isDialogVisible);
+      getCloseButton()?.click();
+      await waitUntil(() => !isDialogVisible());
+      getSecondButton()?.click();
+      await waitUntil(() => !isDialogRendered());
+      getFirstButton()?.click();
+      await waitUntil(isDialogRendered);
+      getInvokerButton()?.click();
+      await waitUntil(isDialogVisible);
+      getCloseButton()?.click();
+      await waitUntil(() => !isDialogVisible());
+      expect(isDialogVisible()).to.equal(false);
+    });
   });
 });

package/components/overlays/src/OverlayMixin.js

@@ -246,16 +246,23 @@ export const OverlayMixinImplementation = superclass =>
 
     /** @protected */
     _setupOverlayCtrl() {
-      if (this._overlayCtrl) return;
-
-      /** @type {OverlayController} */
-      this._overlayCtrl = this._defineOverlay({
+      if (this.#hasSetup) return;
+      const config = {
         contentNode: this._overlayContentNode,
         contentWrapperNode: this._overlayContentWrapperNode,
         invokerNode: this._overlayInvokerNode,
         referenceNode: this._overlayReferenceNode,
         backdropNode: this._overlayBackdropNode,
-      });
+      };
+
+      if (this._overlayCtrl) {
+        // when `lit` `cache` attaches node to the DOM, register the controller back in the OverlaysManager
+        this._overlayCtrl.updateConfig(config);
+      } else {
+        /** @type {OverlayController} */
+        this._overlayCtrl = this._defineOverlay(config);
+      }
+
       this.__syncToOverlayController();
       this.__setupSyncFromOverlayController();
       this._setupOpenCloseListeners();

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

@@ -153,6 +153,8 @@ async function createNestedEscControllers(parentContent) {
 
 afterEach(() => {
   overlays.teardown();
+  // clean document.body from the DOM nodes left by previous tests
+  document.body.innerHTML = '';
 });
 
 describe('OverlayController', () => {

package/components/overlays/test-suites/OverlayMixin.suite.js

@@ -533,8 +533,11 @@ export function runOverlayMixinSuite({ tagString, tag, suffix = '' }) {
 
         el.switched = false;
         await el.updateComplete;
-
         expect(setupSpy.callCount).to.equal(1);
+
+        const elCtrl = /** @type {OverlayEl} */ (el.overlayEl)?._overlayCtrl;
+        const isCtrlRegisteredAtOverlaysManager = elCtrl.manager.list.some(ctrl => elCtrl === ctrl);
+        expect(isCtrlRegisteredAtOverlaysManager).to.equal(true);
       });
 
       it('correctly removes event listeners when disconnected from dom', async () => {

package/custom-elements.json

@@ -2145,6 +2145,36 @@
         }
       ]
     },
+    {
+      "kind": "javascript-module",
+      "path": "exports/define-helpers/sb-action-logger.js",
+      "declarations": [],
+      "exports": [
+        {
+          "kind": "custom-element-definition",
+          "name": "sb-action-logger",
+          "declaration": {
+            "name": "SbActionLogger",
+            "module": "/exports/helpers.js"
+          }
+        }
+      ]
+    },
+    {
+      "kind": "javascript-module",
+      "path": "exports/define-helpers/sb-locale-switcher.js",
+      "declarations": [],
+      "exports": [
+        {
+          "kind": "custom-element-definition",
+          "name": "sb-locale-switcher",
+          "declaration": {
+            "name": "SbLocaleSwitcher",
+            "module": "/exports/helpers.js"
+          }
+        }
+      ]
+    },
     {
       "kind": "javascript-module",
       "path": "exports/define/lion-accordion.js",
@@ -2850,36 +2880,6 @@
         }
       ]
     },
-    {
-      "kind": "javascript-module",
-      "path": "exports/define-helpers/sb-action-logger.js",
-      "declarations": [],
-      "exports": [
-        {
-          "kind": "custom-element-definition",
-          "name": "sb-action-logger",
-          "declaration": {
-            "name": "SbActionLogger",
-            "module": "/exports/helpers.js"
-          }
-        }
-      ]
-    },
-    {
-      "kind": "javascript-module",
-      "path": "exports/define-helpers/sb-locale-switcher.js",
-      "declarations": [],
-      "exports": [
-        {
-          "kind": "custom-element-definition",
-          "name": "sb-locale-switcher",
-          "declaration": {
-            "name": "SbLocaleSwitcher",
-            "module": "/exports/helpers.js"
-          }
-        }
-      ]
-    },
     {
       "kind": "javascript-module",
       "path": "components/accordion/src/LionAccordion.js",

package/dist-types/components/dialog/test/lion-dialog.test.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"lion-dialog.test.d.ts","sourceRoot":"","sources":["../../../../components/dialog/test/lion-dialog.test.js"],"names":[],"mappings":"yBAQa,OAAO,sBAAsB,EAAE,UAAU;6BACzC,OAAO,KAAK,EAAE,cAAc"}
+{"version":3,"file":"lion-dialog.test.d.ts","sourceRoot":"","sources":["../../../../components/dialog/test/lion-dialog.test.js"],"names":[],"mappings":"yBAmBa,OAAO,sBAAsB,EAAE,UAAU;6BACzC,OAAO,KAAK,EAAE,cAAc"}

package/dist-types/components/overlays/src/OverlayMixin.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"OverlayMixin.d.ts","sourceRoot":"","sources":["../../../../components/overlays/src/OverlayMixin.js"],"names":[],"mappings":"AAIA;;;;GAIG;AAEH;;;;;;;GAOG;AACH,yCAHU,YAAY,CA8YlB;2BAtZS,OAAO,+BAA+B,EAAE,YAAY;AAuZjE,gGAAoE;4BAzZvD,OAAO,2BAA2B,EAAE,aAAa;kCACjD,OAAO,+BAA+B,EAAE,mBAAmB"}
+{"version":3,"file":"OverlayMixin.d.ts","sourceRoot":"","sources":["../../../../components/overlays/src/OverlayMixin.js"],"names":[],"mappings":"AAIA;;;;GAIG;AAEH;;;;;;;GAOG;AACH,yCAHU,YAAY,CAqZlB;2BA7ZS,OAAO,+BAA+B,EAAE,YAAY;AA8ZjE,gGAAoE;4BAhavD,OAAO,2BAA2B,EAAE,aAAa;kCACjD,OAAO,+BAA+B,EAAE,mBAAmB"}

package/dist-types/components/overlays/test-suites/OverlayMixin.suite.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"OverlayMixin.suite.d.ts","sourceRoot":"","sources":["../../../../components/overlays/test-suites/OverlayMixin.suite.js"],"names":[],"mappings":"AAwDA;;GAEG;AACH,iEAFW;IAAC,SAAS,EAAC,MAAM,CAAC;IAAC,GAAG,EAAE,MAAM,CAAC;IAAC,MAAM,CAAC,EAAC,MAAM,CAAA;CAAC,QA4mBzD;kCAlpBY,OAAO,+BAA+B,EAAE,mBAAmB;wBAC3D,UAAU,GAAG,WAAW,GAAG;IAAC,YAAY,EAAC,iBAAiB,CAAA;CAAC;2BAC3D,OAAO,+BAA+B,EAAE,YAAY;0BACpD,OAAO,+BAA+B,EAAE,WAAW;4BACnD,OAAO,2BAA2B,EAAE,aAAa"}
+{"version":3,"file":"OverlayMixin.suite.d.ts","sourceRoot":"","sources":["../../../../components/overlays/test-suites/OverlayMixin.suite.js"],"names":[],"mappings":"AAwDA;;GAEG;AACH,iEAFW;IAAC,SAAS,EAAC,MAAM,CAAC;IAAC,GAAG,EAAE,MAAM,CAAC;IAAC,MAAM,CAAC,EAAC,MAAM,CAAA;CAAC,QA+mBzD;kCArpBY,OAAO,+BAA+B,EAAE,mBAAmB;wBAC3D,UAAU,GAAG,WAAW,GAAG;IAAC,YAAY,EAAC,iBAAiB,CAAA;CAAC;2BAC3D,OAAO,+BAA+B,EAAE,YAAY;0BACpD,OAAO,+BAA+B,EAAE,WAAW;4BACnD,OAAO,2BAA2B,EAAE,aAAa"}