@lion/ui 0.5.1 → 0.5.2

package/components/overlays/src/OverlayController.js

@@ -933,7 +933,7 @@ export class OverlayController extends EventTarget {
 
     if (this.elementToFocusAfterHide) {
       this.elementToFocusAfterHide.focus();
-      this.elementToFocusAfterHide.scrollIntoView({ block: 'center' });
+      this.elementToFocusAfterHide.scrollIntoView({ block: 'nearest' });
     } else {
       /** @type {HTMLElement} */ (this.__activeElementRightBeforeHide).blur();
     }

package/components/overlays/src/configurations/visibility-trigger-partials/withHoverInteraction.js

@@ -32,15 +32,13 @@ export function withHoverInteraction({ delayIn = 0, delayOut = 300 }) {
        */
       function handleOpenClosed(event) {
         const { type } = event;
-        if (controller._hasDisabledInvoker()) {
-          return;
-        }
+
         clearTimeout(delayTimeout);
         isFocused = type === 'focusout' ? false : isFocused || type === 'focusin';
         isHovered = type === 'mouseleave' ? false : isHovered || type === 'mouseenter';
         const shouldOpen = isFocused || isHovered;
 
-        if (shouldOpen) {
+        if (shouldOpen && !controller._hasDisabledInvoker()) {
           delayTimeout = setTimeout(() => {
             controller.show();
           }, delayIn);

package/components/tabs/src/LionTabs.js

@@ -356,7 +356,11 @@ export class LionTabs extends LitElement {
    * @param {number} value The new index
    */
   set selectedIndex(value) {
+    if (value === this.__selectedIndex) {
+      return;
+    }
     const stale = this.__selectedIndex;
+    /** @type {number | undefined} */
     this.__selectedIndex = value;
     this.__updateSelected(false);
     this.dispatchEvent(new Event('selected-changed'));

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

@@ -75,6 +75,14 @@ describe('<lion-tabs>', () => {
       expect(spy).to.have.been.calledOnce;
     });
 
+    it('does not send event "selected-changed" after selecting currently selected tab', async () => {
+      const el = /** @type {LionTabs} */ (await fixture(basicTabs));
+      const spy = sinon.spy();
+      el.addEventListener('selected-changed', spy);
+      el.selectedIndex = 0;
+      expect(spy).not.to.have.been.called;
+    });
+
     it('logs warning if unequal amount of tabs and panels', async () => {
       const stub = sinon.stub(console, 'warn');
       await fixture(html`

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

@@ -182,6 +182,36 @@ describe('lion-tooltip', () => {
       expect(el._overlayCtrl.isShown).to.equal(false);
     });
 
+    it('gets hidden when invoker gets disabled', async () => {
+      const el = /** @type {LionTooltip} */ (
+        await fixture(html`
+          <lion-tooltip>
+            <div slot="content">Hey there</div>
+            <button slot="invoker">Tooltip button</button>
+          </lion-tooltip>
+        `)
+      );
+      const invoker = /** @type {HTMLButtonElement} */ (
+        Array.from(el.children).find(child => child.slot === 'invoker')
+      );
+      const eventFocusIn = new Event('focusin');
+      invoker.dispatchEvent(eventFocusIn);
+      clock.tick(300);
+      await el.updateComplete;
+      // @ts-expect-error [allow-protected-in-tests]
+      expect(el._overlayCtrl.isShown).to.equal(true);
+
+      invoker.setAttribute('disabled', '');
+
+      const eventFocusOut = new Event('focusout');
+      invoker.dispatchEvent(eventFocusOut);
+      clock.tick(300);
+      await el.updateComplete;
+      await el.updateComplete; // webkit needs longer
+      // @ts-expect-error [allow-protected-in-tests]
+      expect(el._overlayCtrl.isShown).to.equal(false);
+    });
+
     it('contains html when specified in tooltip content body', async () => {
       const el = /** @type {LionTooltip} */ (
         await fixture(html`

package/custom-elements.json

@@ -63249,12 +63249,6 @@
         }
       ]
     },
-    {
-      "kind": "javascript-module",
-      "path": "components/input-file/types/input-file.ts",
-      "declarations": [],
-      "exports": []
-    },
     {
       "kind": "javascript-module",
       "path": "components/input-file/translations/bg-BG.js",
@@ -63745,6 +63739,12 @@
         }
       ]
     },
+    {
+      "kind": "javascript-module",
+      "path": "components/input-file/types/input-file.ts",
+      "declarations": [],
+      "exports": []
+    },
     {
       "kind": "javascript-module",
       "path": "components/input-iban/src/LionInputIban.js",

package/dist-types/components/overlays/src/configurations/visibility-trigger-partials/withHoverInteraction.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"withHoverInteraction.d.ts","sourceRoot":"","sources":["../../../../../../components/overlays/src/configurations/visibility-trigger-partials/withHoverInteraction.js"],"names":[],"mappings":"AAAA;;;GAGG;AAIH;;;;;;GAMG;AACH,4DAHW;IAAE,OAAO,CAAC,EAAE,MAAM,CAAC;IAAC,QAAQ,CAAC,EAAE,MAAM,CAAA;CAAG,GACtC,QAAQ,aAAa,CAAC,CA+DlC;4BA1EY,OAAO,4BAA4B,EAAE,aAAa;gCAClD,OAAO,sBAAsB,EAAE,iBAAiB"}
+{"version":3,"file":"withHoverInteraction.d.ts","sourceRoot":"","sources":["../../../../../../components/overlays/src/configurations/visibility-trigger-partials/withHoverInteraction.js"],"names":[],"mappings":"AAAA;;;GAGG;AAIH;;;;;;GAMG;AACH,4DAHW;IAAE,OAAO,CAAC,EAAE,MAAM,CAAC;IAAC,QAAQ,CAAC,EAAE,MAAM,CAAA;CAAG,GACtC,QAAQ,aAAa,CAAC,CA6DlC;4BAxEY,OAAO,4BAA4B,EAAE,aAAa;gCAClD,OAAO,sBAAsB,EAAE,iBAAiB"}

package/dist-types/components/tabs/src/LionTabs.d.ts

@@ -52,6 +52,7 @@ export class LionTabs extends LitElement {
      * @private
      */
     private __handleButtonKeyup;
+    /** @type {number | undefined} */
     __selectedIndex: number | undefined;
     /**
      * @param {number} value The new index for focus

package/dist-types/components/tabs/src/LionTabs.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"LionTabs.d.ts","sourceRoot":"","sources":["../../../../components/tabs/src/LionTabs.js"],"names":[],"mappings":"AAqGA;IACE;;;;;;MAQC;IAED,+CA0BC;IAED,+CASC;IA6MD;;OAEG;IACH,+BAMC;IAhBD;;OAEG;IACH,4BAEC;IAxLD,gCAIC;IAED,4BAIC;IAED,eAAe;IACf,qBAaC;IAED,eAAe;IACf,qBAiCC;IAhCC,2BAA2B;IAC3B,kCAAiB;IAiCnB,eAAe;IACf,qBAQC;IAED;;;;;;OAMG;IACH,gCAUC;IAED;;;;;OAKG;IACH,gCAuBC;IAED;;;;OAIG;IACH,mCAIC;IAED;;;OAGG;IACH,4BAqCC;IAcC,oCAA4B;IAM9B;;;OAGG;IACH,4CAHW,MAAM,QAYhB;IAED,iBAAiB;IACjB,mCAEC;IAED,eAAe;IACf,yBAqBC;CACF;;;;;QAnZa,WAAW;;;;SACX,MAAM;;;;YACN,WAAW;;;;WACX,WAAW;;;;0BACH,KAAK,KAAK,OAAO;;;;4BACjB,KAAK,KAAK,OAAO;;;;0BACjB,KAAK,KAAK,OAAO"}
+{"version":3,"file":"LionTabs.d.ts","sourceRoot":"","sources":["../../../../components/tabs/src/LionTabs.js"],"names":[],"mappings":"AAqGA;IACE;;;;;;MAQC;IAED,+CA0BC;IAED,+CASC;IA6MD;;OAEG;IACH,+BAUC;IApBD;;OAEG;IACH,4BAEC;IAxLD,gCAIC;IAED,4BAIC;IAED,eAAe;IACf,qBAaC;IAED,eAAe;IACf,qBAiCC;IAhCC,2BAA2B;IAC3B,kCAAiB;IAiCnB,eAAe;IACf,qBAQC;IAED;;;;;;OAMG;IACH,gCAUC;IAED;;;;;OAKG;IACH,gCAuBC;IAED;;;;OAIG;IACH,mCAIC;IAED;;;OAGG;IACH,4BAqCC;IAiBC,iCAAiC;IACjC,iBADW,MAAM,GAAG,SAAS,CACD;IAM9B;;;OAGG;IACH,4CAHW,MAAM,QAYhB;IAED,iBAAiB;IACjB,mCAEC;IAED,eAAe;IACf,yBAqBC;CACF;;;;;QAvZa,WAAW;;;;SACX,MAAM;;;;YACN,WAAW;;;;WACX,WAAW;;;;0BACH,KAAK,KAAK,OAAO;;;;4BACjB,KAAK,KAAK,OAAO;;;;0BACjB,KAAK,KAAK,OAAO"}