@lion/ui 0.14.2 → 0.14.3

package/components/select-rich/src/LionSelectRich.js

@@ -146,8 +146,10 @@ export class LionSelectRich extends SlotMixin(ScopedElementsMixin(OverlayMixin(L
 
   connectedCallback() {
     super.connectedCallback();
-    this._invokerNode.selectedElement =
-      this.formElements[/**  @type {number} */ (this.checkedIndex)];
+    this.registrationComplete.then(() => {
+      this._invokerNode.selectedElement =
+        this.formElements[/**  @type {number} */ (this.checkedIndex)];
+    });
 
     this._invokerNode.hostElement = this;
 

package/components/select-rich/test/lion-select-rich.test.js

@@ -6,7 +6,8 @@ import { LionOption } from '@lion/ui/listbox.js';
 import '@lion/ui/define/lion-select-rich.js';
 import '@lion/ui/define/lion-listbox.js';
 import '@lion/ui/define/lion-option.js';
-import { LitElement } from 'lit';
+import '@lion/ui/define/lion-tabs.js';
+import { LitElement, nothing } from 'lit';
 import sinon from 'sinon';
 import {
   fixture as _fixture,
@@ -16,7 +17,9 @@ import {
   defineCE,
   expect,
   html,
+  waitUntil,
 } from '@open-wc/testing';
+import { cache } from 'lit/directives/cache.js';
 
 import { isActiveElement } from '../../core/test-helpers/isActiveElement.js';
 
@@ -724,6 +727,378 @@ describe('lion-select-rich', () => {
     });
   });
 
+  describe('Render from `lit` `cache`', () => {
+    describe('when `lion-option` is set by `.choiceValue` property with and no `checked` attribute', () => {
+      it('should display the second option', async () => {
+        const colours = [
+          {
+            label: 'Red',
+            value: 'red',
+          },
+          {
+            label: 'Blue',
+            value: 'blue',
+          },
+        ];
+
+        /**
+         * 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)}>
+                  Info
+                </button>
+                <p slot="panel">
+                  ${cache(
+                    this.activeTabIndex === 0
+                      ? html`<lion-select-rich name="favoriteColor" label="Favorite color">
+                          ${colours.map(
+                            colour =>
+                              html`<lion-option .choiceValue="${colour.value}"
+                                >${colour.label}</lion-option
+                              >`,
+                          )}
+                        </lion-select-rich>`
+                      : nothing,
+                  )}
+                </p>
+                <button slot="tab" class="second-button" @click=${() => changeActiveTabIndexRef(1)}>
+                  Work
+                </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 getInvoker = () => wrapperElementShadowRoot?.querySelector('lion-select-invoker');
+        /**
+         * @returns { boolean }
+         */
+        const isSelectRichRendered = () => !!getInvoker()?.shadowRoot?.childNodes.length;
+        /**
+         * @returns { string | undefined }
+         */
+        const getSelectedColourLabel = () =>
+          getInvoker()?.shadowRoot?.querySelector('#content-wrapper')?.textContent?.trim();
+        const getDialog = () =>
+          wrapperElementShadowRoot
+            ?.querySelector('lion-select-rich')
+            ?.shadowRoot?.querySelector('dialog');
+        const isDialogVisible = () =>
+          // @ts-ignore
+          getDialog()?.checkVisibility() === true &&
+          getDialog()?.querySelector('#overlay-content-node-wrapper[data-popper-placement]');
+        await waitUntil(isSelectRichRendered);
+        getInvoker()?.click();
+        await waitUntil(isDialogVisible);
+        const optionBlue = /** @type { HTMLElement | undefined } */ (
+          wrapperElementShadowRoot?.querySelectorAll('lion-option')
+        )?.[1];
+        optionBlue?.click();
+        await waitUntil(() => !isDialogVisible());
+        const selectedColourLabelBeforeTabSwitch = getSelectedColourLabel();
+        expect(selectedColourLabelBeforeTabSwitch).to.equal('Blue');
+        getSecondButton()?.click();
+        await waitUntil(() => !isSelectRichRendered());
+        getFirstButton()?.click();
+        await waitUntil(isSelectRichRendered);
+        const selectedColourAfterTabSwitch = getSelectedColourLabel();
+        expect(selectedColourAfterTabSwitch).to.equal('Blue');
+      });
+    });
+    describe('when `lion-option` is set by `.choiceValue` property and `checked` attribute', () => {
+      it('should display the second option', async () => {
+        const colours = [
+          {
+            label: 'Red',
+            value: 'red',
+            checked: true,
+          },
+          {
+            label: 'Blue',
+            value: 'blue',
+            checked: false,
+          },
+        ];
+
+        /**
+         * 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)}>
+                  Info
+                </button>
+                <p slot="panel">
+                  ${cache(
+                    this.activeTabIndex === 0
+                      ? html`<lion-select-rich name="favoriteColor" label="Favorite color">
+                          ${colours.map(
+                            colour =>
+                              html`<lion-option
+                                .choiceValue="${colour.value}"
+                                checked="${colour.checked || nothing}"
+                                >${colour.label}</lion-option
+                              >`,
+                          )}
+                        </lion-select-rich>`
+                      : nothing,
+                  )}
+                </p>
+                <button slot="tab" class="second-button" @click=${() => changeActiveTabIndexRef(1)}>
+                  Work
+                </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 getInvoker = () => wrapperElementShadowRoot?.querySelector('lion-select-invoker');
+        /**
+         * @returns { boolean }
+         */
+        const isSelectRichRendered = () => !!getInvoker()?.shadowRoot?.childNodes.length;
+        /**
+         * @returns { string | undefined }
+         */
+        const getSelectedColourLabel = () =>
+          getInvoker()?.shadowRoot?.querySelector('#content-wrapper')?.textContent?.trim();
+        const getDialog = () =>
+          wrapperElementShadowRoot
+            ?.querySelector('lion-select-rich')
+            ?.shadowRoot?.querySelector('dialog');
+        const isDialogVisible = () =>
+          // @ts-ignore
+          getDialog()?.checkVisibility() === true &&
+          getDialog()?.querySelector('#overlay-content-node-wrapper[data-popper-placement]');
+        await waitUntil(isSelectRichRendered);
+        getInvoker()?.click();
+        await waitUntil(isDialogVisible);
+        const optionBlue = /** @type { HTMLElement | undefined } */ (
+          wrapperElementShadowRoot?.querySelectorAll('lion-option')
+        )?.[1];
+        optionBlue?.click();
+        await waitUntil(() => !isDialogVisible());
+        const selectedColourLabelBeforeTabSwitch = getSelectedColourLabel();
+        expect(selectedColourLabelBeforeTabSwitch).to.equal('Blue');
+        getSecondButton()?.click();
+        await waitUntil(() => !isSelectRichRendered());
+        getFirstButton()?.click();
+        await waitUntil(isSelectRichRendered);
+        const selectedColourAfterTabSwitch = getSelectedColourLabel();
+        expect(selectedColourAfterTabSwitch).to.equal('Blue');
+      });
+    });
+    describe('when `lion-option` is set by `.modelValue` property', () => {
+      it('should display the second option', async () => {
+        const colours = [
+          {
+            label: 'Red',
+            value: 'red',
+            checked: true,
+          },
+          {
+            label: 'Blue',
+            value: 'blue',
+            checked: false,
+          },
+        ];
+
+        /**
+         * 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)}>
+                  Info
+                </button>
+                <p slot="panel">
+                  ${cache(
+                    this.activeTabIndex === 0
+                      ? html`<lion-select-rich name="favoriteColor" label="Favorite color">
+                          ${colours.map(
+                            colour =>
+                              html`<lion-option
+                                @model-value-changed=${(/** @type {Event} */ ev) => {
+                                  const lionOption = /** @type {LionOption} */ (ev.target);
+                                  // eslint-disable-next-line no-param-reassign
+                                  colour.checked = lionOption.modelValue.checked;
+                                }}
+                                .modelValue=${{ value: colour.value, checked: colour.checked }}
+                                >${colour.label}</lion-option
+                              >`,
+                          )}
+                        </lion-select-rich>`
+                      : nothing,
+                  )}
+                </p>
+                <button slot="tab" class="second-button" @click=${() => changeActiveTabIndexRef(1)}>
+                  Work
+                </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 getInvoker = () => wrapperElementShadowRoot?.querySelector('lion-select-invoker');
+        /**
+         * @returns { boolean }
+         */
+        const isSelectRichRendered = () => !!getInvoker()?.shadowRoot?.childNodes.length;
+        /**
+         * @returns { string | undefined }
+         */
+        const getSelectedColourLabel = () =>
+          getInvoker()?.shadowRoot?.querySelector('#content-wrapper')?.textContent?.trim();
+        const getDialog = () =>
+          wrapperElementShadowRoot
+            ?.querySelector('lion-select-rich')
+            ?.shadowRoot?.querySelector('dialog');
+        const isDialogVisible = () =>
+          // @ts-ignore
+          getDialog()?.checkVisibility() === true &&
+          getDialog()?.querySelector('#overlay-content-node-wrapper[data-popper-placement]');
+        await waitUntil(isSelectRichRendered);
+        getInvoker()?.click();
+        await waitUntil(isDialogVisible);
+        const optionBlue = /** @type { HTMLElement | undefined } */ (
+          wrapperElementShadowRoot?.querySelectorAll('lion-option')
+        )?.[1];
+        optionBlue?.click();
+        await waitUntil(() => !isDialogVisible());
+        const selectedColourLabelBeforeTabSwitch = getSelectedColourLabel();
+        expect(selectedColourLabelBeforeTabSwitch).to.equal('Blue');
+        getSecondButton()?.click();
+        await waitUntil(() => !isSelectRichRendered());
+        getFirstButton()?.click();
+        await waitUntil(isSelectRichRendered);
+        const selectedColourAfterTabSwitch = getSelectedColourLabel();
+        expect(selectedColourAfterTabSwitch).to.equal('Blue');
+      });
+    });
+  });
+
   describe('Teardown', () => {
     it('correctly removes event listeners when disconnected from dom', async () => {
       const el = await fixture(html`

package/dist-types/components/select-rich/src/LionSelectRich.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"LionSelectRich.d.ts","sourceRoot":"","sources":["../../../../components/select-rich/src/LionSelectRich.js"],"names":[],"mappings":";AAsBA;;;;GAIG;AACH;IACE;;MAKC;IAED,kBAAkB;IAClB,6BAgBC;IAED;;;OAGG;IAEH,2EAUC;IAUD;;;OAGG;IACH,gDAIC;IAED;;;OAGG;IAEH,kDAEC;IAiBC;;;;OAIG;IACH,+BAAkC;IAClC;;;;OAIG;IACH,iBAFU,eAAe,GAAC,KAAK,GAAC,MAAM,CAET;IAE7B,sBAAyB;IACzB,iBAAiB;IACjB,8BAAqB;IAyWvB;;;OAGG;IAEH,wBAJW,aAAa,QAgEvB;IA9ND,eAAe;IACf,wBAEC;IAiED,eAAe;IACf,wBAGC;IAZD,eAAe;IACf,wBAKC;IAzED,eAAe;IACf,yBAIC;IAkDD,eAAe;IACf,4BASC;IA2MD,iBAAiB;IACjB,kCAEC;IAlcD,0BAUC;IAQD;;;;;OAKG;IACH,oDAJW,OAAO,sEAEL,IAAI,CAgBhB;IAED;;OAEG;IACH,2BAFW,OAAO,KAAK,EAAE,cAAc,QAiDtC;IAED;;;;OAIG;IACH,sBAHW,UAAU,GAAG,eAAe,iCAoBtC;IALG,qDAA2C;IAiB/C;;;;;OAKG;IACH,iGAEC;IAED,iBAAiB;IACjB,yCAGC;IAED,eAAe;IACf,gCAEC;IAED,eAAe;IACf,gCAKC;IAED,eAAe;IACf,6BAaC;IAED,eAAe;IACf,2BAKC;IAcD,eAAe;IACf,wCAIC;IAED,eAAe;IACf,8BAGC;IAED;;;OAGG;IAEH;;;;;;;;;;;;;;;;;;;;;;;;;MAMC;IAED;;;;;;OAMG;IACH,kDAQC;IAkCC,4EAA8E;IAqBhF;;;;OAIG;IACH,8CAyBC;IAED;;;OAGG;IACH,gCAEC;IAED;;;OAGG;IACH,uDAEC;IAED;;;OAGG;IACH,4FAEC;IAgBC,0CAAgC;IA6GlC;;;;OAIG;IACH,wCASC;IARC,qCAA2B;CAS9B;0BA3mBY,OAAO,kCAAkC,EAAE,WAAW;yBACtD,OAAO,iCAAiC,EAAE,UAAU;kCACpD,OAAO,iEAAiE,EAAE,mBAAmB;8BAC7F,OAAO,gDAAgD,EAAE,eAAe;uBACxE,OAAO,oCAAoC,EAAE,QAAQ"}
+{"version":3,"file":"LionSelectRich.d.ts","sourceRoot":"","sources":["../../../../components/select-rich/src/LionSelectRich.js"],"names":[],"mappings":";AAsBA;;;;GAIG;AACH;IACE;;MAKC;IAED,kBAAkB;IAClB,6BAgBC;IAED;;;OAGG;IAEH,2EAUC;IAUD;;;OAGG;IACH,gDAIC;IAED;;;OAGG;IAEH,kDAEC;IAiBC;;;;OAIG;IACH,+BAAkC;IAClC;;;;OAIG;IACH,iBAFU,eAAe,GAAC,KAAK,GAAC,MAAM,CAET;IAE7B,sBAAyB;IACzB,iBAAiB;IACjB,8BAAqB;IA2WvB;;;OAGG;IAEH,wBAJW,aAAa,QAgEvB;IA9ND,eAAe;IACf,wBAEC;IAiED,eAAe;IACf,wBAGC;IAZD,eAAe;IACf,wBAKC;IAzED,eAAe;IACf,yBAIC;IAkDD,eAAe;IACf,4BASC;IA2MD,iBAAiB;IACjB,kCAEC;IApcD,0BAYC;IAQD;;;;;OAKG;IACH,oDAJW,OAAO,sEAEL,IAAI,CAgBhB;IAED;;OAEG;IACH,2BAFW,OAAO,KAAK,EAAE,cAAc,QAiDtC;IAED;;;;OAIG;IACH,sBAHW,UAAU,GAAG,eAAe,iCAoBtC;IALG,qDAA2C;IAiB/C;;;;;OAKG;IACH,iGAEC;IAED,iBAAiB;IACjB,yCAGC;IAED,eAAe;IACf,gCAEC;IAED,eAAe;IACf,gCAKC;IAED,eAAe;IACf,6BAaC;IAED,eAAe;IACf,2BAKC;IAcD,eAAe;IACf,wCAIC;IAED,eAAe;IACf,8BAGC;IAED;;;OAGG;IAEH;;;;;;;;;;;;;;;;;;;;;;;;;MAMC;IAED;;;;;;OAMG;IACH,kDAQC;IAkCC,4EAA8E;IAqBhF;;;;OAIG;IACH,8CAyBC;IAED;;;OAGG;IACH,gCAEC;IAED;;;OAGG;IACH,uDAEC;IAED;;;OAGG;IACH,4FAEC;IAgBC,0CAAgC;IA6GlC;;;;OAIG;IACH,wCASC;IARC,qCAA2B;CAS9B;0BA7mBY,OAAO,kCAAkC,EAAE,WAAW;yBACtD,OAAO,iCAAiC,EAAE,UAAU;kCACpD,OAAO,iEAAiE,EAAE,mBAAmB;8BAC7F,OAAO,gDAAgD,EAAE,eAAe;uBACxE,OAAO,oCAAoC,EAAE,QAAQ"}

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

@@ -1 +1 @@
-{"version":3,"file":"lion-select-rich.test.d.ts","sourceRoot":"","sources":["../../../../components/select-rich/test/lion-select-rich.test.js"],"names":[],"mappings":"8BAuBa,OAAO,gDAAgD,EAAE,eAAe;0BACxE,OAAO,0CAA0C,EAAE,WAAW;0BAC9D,OAAO,kCAAkC,EAAE,WAAW;6BACtD,OAAO,KAAK,EAAE,cAAc"}
+{"version":3,"file":"lion-select-rich.test.d.ts","sourceRoot":"","sources":["../../../../components/select-rich/test/lion-select-rich.test.js"],"names":[],"mappings":"8BA0Ba,OAAO,gDAAgD,EAAE,eAAe;0BACxE,OAAO,0CAA0C,EAAE,WAAW;0BAC9D,OAAO,kCAAkC,EAAE,WAAW;6BACtD,OAAO,KAAK,EAAE,cAAc"}