@vaadin/context-menu 23.1.0-alpha1 → 23.1.0-alpha4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@vaadin/context-menu",
-  "version": "23.1.0-alpha1",
+  "version": "23.1.0-alpha4",
   "publishConfig": {
     "access": "public"
   },
@@ -35,19 +35,19 @@
   "dependencies": {
     "@open-wc/dedupe-mixin": "^1.3.0",
     "@polymer/polymer": "^3.0.0",
-    "@vaadin/component-base": "23.1.0-alpha1",
-    "@vaadin/item": "23.1.0-alpha1",
-    "@vaadin/list-box": "23.1.0-alpha1",
-    "@vaadin/vaadin-lumo-styles": "23.1.0-alpha1",
-    "@vaadin/vaadin-material-styles": "23.1.0-alpha1",
-    "@vaadin/vaadin-overlay": "23.1.0-alpha1",
-    "@vaadin/vaadin-themable-mixin": "23.1.0-alpha1"
+    "@vaadin/component-base": "23.1.0-alpha4",
+    "@vaadin/item": "23.1.0-alpha4",
+    "@vaadin/list-box": "23.1.0-alpha4",
+    "@vaadin/vaadin-lumo-styles": "23.1.0-alpha4",
+    "@vaadin/vaadin-material-styles": "23.1.0-alpha4",
+    "@vaadin/vaadin-overlay": "23.1.0-alpha4",
+    "@vaadin/vaadin-themable-mixin": "23.1.0-alpha4"
   },
   "devDependencies": {
     "@esm-bundle/chai": "^4.3.4",
-    "@vaadin/polymer-legacy-adapter": "23.1.0-alpha1",
+    "@vaadin/polymer-legacy-adapter": "23.1.0-alpha4",
     "@vaadin/testing-helpers": "^0.3.2",
-    "sinon": "^9.2.1"
+    "sinon": "^13.0.2"
   },
-  "gitHead": "5d0cdee069f866037c507265fafb4d0476795333"
+  "gitHead": "aacdb7fe09811894751f0378ff7fb66071892c71"
 }

package/src/vaadin-context-menu-overlay.js

@@ -28,7 +28,7 @@ registerStyles(
       background-color: #fff;
     }
   `,
-  { moduleId: 'vaadin-context-menu-overlay-styles' }
+  { moduleId: 'vaadin-context-menu-overlay-styles' },
 );
 
 /**
@@ -49,8 +49,8 @@ class ContextMenuOverlay extends PositionMixin(OverlayElement) {
        */
       parentOverlay: {
         type: Object,
-        readOnly: true
-      }
+        readOnly: true,
+      },
     };
   }
 
@@ -102,7 +102,7 @@ class ContextMenuOverlay extends PositionMixin(OverlayElement) {
     return {
       xMax: overlayRect.right - contentRect.width,
       xMin: overlayRect.left + contentRect.width,
-      yMax
+      yMax,
     };
   }
 
@@ -118,17 +118,17 @@ class ContextMenuOverlay extends PositionMixin(OverlayElement) {
       // Horizontal adjustment
       const isLeftAligned = !!this.style.left;
       if (isLeftAligned) {
-        this.style.left = parseFloat(this.style.left) + parseFloat(style.paddingLeft) + 'px';
+        this.style.left = `${parseFloat(this.style.left) + parseFloat(style.paddingLeft)}px`;
       } else {
-        this.style.right = parseFloat(this.style.right) + parseFloat(style.paddingRight) + 'px';
+        this.style.right = `${parseFloat(this.style.right) + parseFloat(style.paddingRight)}px`;
       }
 
       // Vertical adjustment
       const isBottomAligned = !!this.style.bottom;
       if (isBottomAligned) {
-        this.style.bottom = parseFloat(this.style.bottom) - parseFloat(style.paddingBottom) + 'px';
+        this.style.bottom = `${parseFloat(this.style.bottom) - parseFloat(style.paddingBottom)}px`;
       } else {
-        this.style.top = parseFloat(this.style.top) - parseFloat(style.paddingTop) + 'px';
+        this.style.top = `${parseFloat(this.style.top) - parseFloat(style.paddingTop)}px`;
       }
     }
   }

package/src/vaadin-context-menu.d.ts

@@ -17,7 +17,7 @@ export interface ContextMenuRendererContext {
 export type ContextMenuRenderer = (
   root: HTMLElement,
   contextMenu?: ContextMenu,
-  context?: ContextMenuRendererContext
+  context?: ContextMenuRendererContext,
 ) => void;
 
 /**
@@ -289,13 +289,13 @@ declare class ContextMenu extends ElementMixin(ThemePropertyMixin(ItemsMixin(HTM
   addEventListener<K extends keyof ContextMenuEventMap>(
     type: K,
     listener: (this: ContextMenu, ev: ContextMenuEventMap[K]) => void,
-    options?: boolean | AddEventListenerOptions
+    options?: boolean | AddEventListenerOptions,
   ): void;
 
   removeEventListener<K extends keyof ContextMenuEventMap>(
     type: K,
     listener: (this: ContextMenu, ev: ContextMenuEventMap[K]) => void,
-    options?: boolean | EventListenerOptions
+    options?: boolean | EventListenerOptions,
   ): void;
 }
 

package/src/vaadin-context-menu.js

@@ -241,7 +241,7 @@ class ContextMenu extends ControllerMixin(ElementMixin(ThemePropertyMixin(ItemsM
        * of the context menu to listen for `openOn` events.
        */
       selector: {
-        type: String
+        type: String,
       },
 
       /**
@@ -252,7 +252,7 @@ class ContextMenu extends ControllerMixin(ElementMixin(ThemePropertyMixin(ItemsM
         type: Boolean,
         value: false,
         notify: true,
-        readOnly: true
+        readOnly: true,
       },
 
       /**
@@ -262,7 +262,7 @@ class ContextMenu extends ControllerMixin(ElementMixin(ThemePropertyMixin(ItemsM
        */
       openOn: {
         type: String,
-        value: 'vaadin-contextmenu'
+        value: 'vaadin-contextmenu',
       },
 
       /**
@@ -276,7 +276,7 @@ class ContextMenu extends ControllerMixin(ElementMixin(ThemePropertyMixin(ItemsM
         type: Object,
         value: function () {
           return this;
-        }
+        },
       },
 
       /**
@@ -287,7 +287,7 @@ class ContextMenu extends ControllerMixin(ElementMixin(ThemePropertyMixin(ItemsM
       closeOn: {
         type: String,
         value: 'click',
-        observer: '_closeOnChanged'
+        observer: '_closeOnChanged',
       },
 
       /**
@@ -302,7 +302,7 @@ class ContextMenu extends ControllerMixin(ElementMixin(ThemePropertyMixin(ItemsM
        * @type {ContextMenuRenderer | undefined}
        */
       renderer: {
-        type: Function
+        type: Function,
       },
 
       /** @private */
@@ -316,25 +316,25 @@ class ContextMenu extends ControllerMixin(ElementMixin(ThemePropertyMixin(ItemsM
 
       /** @private */
       _phone: {
-        type: Boolean
+        type: Boolean,
       },
 
       /** @private */
       _touch: {
         type: Boolean,
-        value: isTouch
+        value: isTouch,
       },
 
       /** @private */
       _wide: {
-        type: Boolean
+        type: Boolean,
       },
 
       /** @private */
       _wideMediaQuery: {
         type: String,
-        value: '(min-device-width: 750px)'
-      }
+        value: '(min-device-width: 750px)',
+      },
     };
   }
 
@@ -343,7 +343,7 @@ class ContextMenu extends ControllerMixin(ElementMixin(ThemePropertyMixin(ItemsM
       '_openedChanged(opened)',
       '_targetOrOpenOnChanged(listenOn, openOn)',
       '_rendererChanged(renderer, items)',
-      '_touchOrWideChanged(_touch, _wide)'
+      '_touchOrWideChanged(_touch, _wide)',
     ];
   }
 
@@ -377,7 +377,7 @@ class ContextMenu extends ControllerMixin(ElementMixin(ThemePropertyMixin(ItemsM
     this.addController(
       new MediaQueryController(this._wideMediaQuery, (matches) => {
         this._wide = matches;
-      })
+      }),
     );
 
     processTemplates(this);
@@ -535,7 +535,7 @@ class ContextMenu extends ControllerMixin(ElementMixin(ThemePropertyMixin(ItemsM
     if (e && !this.opened) {
       this._context = {
         detail: e.detail,
-        target: this._contextTarget(e)
+        target: this._contextTarget(e),
       };
 
       if (this._context.target) {
@@ -579,7 +579,7 @@ class ContextMenu extends ControllerMixin(ElementMixin(ThemePropertyMixin(ItemsM
     const overlay = this.$.overlay;
     const style = overlay.style;
 
-    style[coord] = (parseInt(style[coord]) || 0) + diff + 'px';
+    style[coord] = `${(parseInt(style[coord]) || 0) + diff}px`;
   }
 
   /** @private */
@@ -618,25 +618,25 @@ class ContextMenu extends ControllerMixin(ElementMixin(ThemePropertyMixin(ItemsM
     if (!this.__isRTL) {
       if (x < wdthVport / 2 || x < xMax) {
         // Menu is displayed in the right side of the anchor
-        style.left = x + 'px';
+        style.left = `${x}px`;
       } else {
         // Menu is displayed in the left side of the anchor
-        style.right = Math.max(0, wdthVport - x) + 'px';
+        style.right = `${Math.max(0, wdthVport - x)}px`;
         this._setEndAligned(overlay);
       }
     } else if (x > wdthVport / 2 || x > xMin) {
       // Menu is displayed in the right side of the anchor
-      style.right = Math.max(0, wdthVport - x) + 'px';
+      style.right = `${Math.max(0, wdthVport - x)}px`;
     } else {
       // Menu is displayed in the left side of the anchor
-      style.left = x + 'px';
+      style.left = `${x}px`;
       this._setEndAligned(overlay);
     }
 
     if (y < hghtVport / 2 || y < yMax) {
-      style.top = y + 'px';
+      style.top = `${y}px`;
     } else {
-      style.bottom = Math.max(0, hghtVport - y) + 'px';
+      style.bottom = `${Math.max(0, hghtVport - y)}px`;
       overlay.setAttribute('bottom-aligned', '');
     }
   }
@@ -660,7 +660,7 @@ class ContextMenu extends ControllerMixin(ElementMixin(ThemePropertyMixin(ItemsM
         return this._getEventCoordinate(event.detail.sourceEvent, coord);
       }
     } else {
-      const prop = 'client' + coord.toUpperCase();
+      const prop = `client${coord.toUpperCase()}`;
       const position = event.changedTouches ? event.changedTouches[0][prop] : event[prop];
 
       if (position === 0) {

package/src/vaadin-contextmenu-event.js

@@ -11,13 +11,13 @@ register({
   deps: ['touchstart', 'touchmove', 'touchend', 'contextmenu'],
   flow: {
     start: ['touchstart', 'contextmenu'],
-    end: ['contextmenu']
+    end: ['contextmenu'],
   },
 
   emits: ['vaadin-contextmenu'],
 
   info: {
-    sourceEvent: null
+    sourceEvent: null,
   },
 
   reset: function () {
@@ -38,7 +38,7 @@ register({
     this.info.sourceEvent = e;
     this.info.touchStartCoords = {
       x: e.changedTouches[0].clientX,
-      y: e.changedTouches[0].clientY
+      y: e.changedTouches[0].clientY,
     };
 
     // After timeout event is already retargeted to the parent element in case there is one.
@@ -97,5 +97,5 @@ register({
     if (ev.defaultPrevented && sourceEvent && sourceEvent.preventDefault) {
       sourceEvent.preventDefault();
     }
-  }
+  },
 });

package/src/vaadin-contextmenu-items-mixin.js

@@ -95,7 +95,7 @@ export const ItemsMixin = (superClass) =>
          * ----------------|----------------|----------------
          * `:host` | expanded | Expanded parent item
          */
-        items: Array
+        items: Array,
       };
     }
 
@@ -178,9 +178,9 @@ export const ItemsMixin = (superClass) =>
       itemElement.dispatchEvent(
         new CustomEvent('opensubmenu', {
           detail: {
-            children: itemElement._item.children
-          }
-        })
+            children: itemElement._item.children,
+          },
+        }),
       );
     }
 
@@ -263,10 +263,10 @@ export const ItemsMixin = (superClass) =>
     __toggleMenuComponentAttribute(component, attribute, on) {
       if (on) {
         component.setAttribute(attribute, '');
-        component['__has-' + attribute] = true;
-      } else if (component['__has-' + attribute]) {
+        component[`__has-${attribute}`] = true;
+      } else if (component[`__has-${attribute}`]) {
         component.removeAttribute(attribute);
-        component['__has-' + attribute] = false;
+        component[`__has-${attribute}`] = false;
       }
     }
 
@@ -280,7 +280,9 @@ export const ItemsMixin = (superClass) =>
       `;
         flush();
         const listBox = root.querySelector('vaadin-context-menu-list-box');
-        this._theme && listBox.setAttribute('theme', this._theme);
+        if (this._theme) {
+          listBox.setAttribute('theme', this._theme);
+        }
         listBox.classList.add('vaadin-menu-list-box');
         requestAnimationFrame(() => listBox.setAttribute('role', 'menu'));
 
@@ -334,7 +336,7 @@ export const ItemsMixin = (superClass) =>
         requestAnimationFrame(() => (this.__openListenerActive = true));
         const openSubMenu = (
           e,
-          itemElement = e.composedPath().filter((e) => e.localName === 'vaadin-context-menu-item')[0]
+          itemElement = e.composedPath().filter((e) => e.localName === 'vaadin-context-menu-item')[0],
         ) => {
           // Delay enabling the mouseover listener to avoid it from triggering on parent menu open
           if (!this.__openListenerActive) {
@@ -373,7 +375,9 @@ export const ItemsMixin = (superClass) =>
           const shouldOpenSubMenu =
             (!isRTL && e.keyCode === 39) || (isRTL && e.keyCode === 37) || e.keyCode === 13 || e.keyCode === 32;
 
-          shouldOpenSubMenu && openSubMenu(e);
+          if (shouldOpenSubMenu) {
+            openSubMenu(e);
+          }
         });
       } else {
         const listBox = root.querySelector('vaadin-context-menu-list-box');

package/theme/lumo/vaadin-context-menu-styles.js

@@ -31,7 +31,7 @@ const contextMenuOverlay = css`
 `;
 
 registerStyles('vaadin-context-menu-overlay', [menuOverlay, contextMenuOverlay], {
-  moduleId: 'lumo-context-menu-overlay'
+  moduleId: 'lumo-context-menu-overlay',
 });
 
 registerStyles(
@@ -84,7 +84,7 @@ registerStyles(
       }
     }
   `,
-  { moduleId: 'lumo-context-menu-list-box' }
+  { moduleId: 'lumo-context-menu-list-box' },
 );
 
 registerStyles(
@@ -124,5 +124,5 @@ registerStyles(
       padding-right: var(--lumo-space-m);
     }
   `,
-  { moduleId: 'lumo-context-menu-item' }
+  { moduleId: 'lumo-context-menu-item' },
 );

package/theme/material/vaadin-context-menu-styles.js

@@ -12,7 +12,7 @@ const contextMenuOverlay = css`
 `;
 
 registerStyles('vaadin-context-menu-overlay', [menuOverlay, contextMenuOverlay], {
-  moduleId: 'material-context-menu-overlay'
+  moduleId: 'material-context-menu-overlay',
 });
 
 registerStyles(
@@ -44,7 +44,7 @@ registerStyles(
       }
     }
   `,
-  { moduleId: 'material-context-menu-list-box' }
+  { moduleId: 'material-context-menu-list-box' },
 );
 
 registerStyles(
@@ -79,5 +79,5 @@ registerStyles(
       background-color: var(--material-secondary-background-color);
     }
   `,
-  { moduleId: 'material-context-menu-item' }
+  { moduleId: 'material-context-menu-item' },
 );