@vaadin/context-menu 23.1.0 → 23.1.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@vaadin/context-menu",
-  "version": "23.1.0",
+  "version": "23.1.3",
   "publishConfig": {
     "access": "public"
   },
@@ -37,20 +37,20 @@
   "dependencies": {
     "@open-wc/dedupe-mixin": "^1.3.0",
     "@polymer/polymer": "^3.0.0",
-    "@vaadin/component-base": "^23.1.0",
-    "@vaadin/item": "^23.1.0",
-    "@vaadin/list-box": "^23.1.0",
-    "@vaadin/lit-renderer": "^23.1.0",
-    "@vaadin/vaadin-lumo-styles": "^23.1.0",
-    "@vaadin/vaadin-material-styles": "^23.1.0",
-    "@vaadin/vaadin-overlay": "^23.1.0",
-    "@vaadin/vaadin-themable-mixin": "^23.1.0"
+    "@vaadin/component-base": "^23.1.3",
+    "@vaadin/item": "^23.1.3",
+    "@vaadin/list-box": "^23.1.3",
+    "@vaadin/lit-renderer": "^23.1.3",
+    "@vaadin/vaadin-lumo-styles": "^23.1.3",
+    "@vaadin/vaadin-material-styles": "^23.1.3",
+    "@vaadin/vaadin-overlay": "^23.1.3",
+    "@vaadin/vaadin-themable-mixin": "^23.1.3"
   },
   "devDependencies": {
     "@esm-bundle/chai": "^4.3.4",
-    "@vaadin/polymer-legacy-adapter": "^23.1.0",
+    "@vaadin/polymer-legacy-adapter": "^23.1.3",
     "@vaadin/testing-helpers": "^0.3.2",
     "sinon": "^13.0.2"
   },
-  "gitHead": "322bba42b83f908a78cd972b06acadc5da95a69d"
+  "gitHead": "3066c296ad0ef652bc49417005523398199f1bf2"
 }

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

@@ -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?: AddEventListenerOptions | boolean,
   ): void;
 
   removeEventListener<K extends keyof ContextMenuEventMap>(
     type: K,
     listener: (this: ContextMenu, ev: ContextMenuEventMap[K]) => void,
-    options?: boolean | EventListenerOptions,
+    options?: EventListenerOptions | boolean,
   ): void;
 }
 

package/src/vaadin-contextmenu-event.js

@@ -34,8 +34,20 @@ register({
     }
   },
 
-  touchstart(e) {
+  _setSourceEvent(e) {
     this.info.sourceEvent = e;
+
+    const path = e.composedPath();
+
+    // Calling `sourceEvent.composedPath()` after a timeout would return an empty array.
+    // This is especially problematic on iOS where we configure the timer on touchstart.
+    // Store the composed path to be used by `grid.getEventContext(event)` so it works.
+    this.info.sourceEvent.__composedPath = path;
+  },
+
+  touchstart(e) {
+    this._setSourceEvent(e);
+
     this.info.touchStartCoords = {
       x: e.changedTouches[0].clientX,
       y: e.changedTouches[0].clientY,
@@ -80,7 +92,7 @@ register({
 
   contextmenu(e) {
     if (!e.shiftKey) {
-      this.info.sourceEvent = e;
+      this._setSourceEvent(e);
       this.fire(e.target, e.clientX, e.clientY);
       prevent('tap');
     }

package/src/vaadin-contextmenu-items-mixin.d.ts

@@ -9,10 +9,10 @@ import { ListBox } from '@vaadin/list-box/src/vaadin-list-box.js';
 
 export interface ContextMenuItem {
   text?: string;
-  component?: string | HTMLElement;
+  component?: HTMLElement | string;
   disabled?: boolean;
   checked?: boolean;
-  theme?: string | string[];
+  theme?: string[] | string;
   children?: ContextMenuItem[];
 }
 
@@ -37,7 +37,7 @@ declare global {
  */
 declare class ContextMenuListBox extends ListBox {}
 
-export declare function ItemsMixin<T extends Constructor<HTMLElement>>(base: T): T & Constructor<ItemsMixinClass>;
+export declare function ItemsMixin<T extends Constructor<HTMLElement>>(base: T): Constructor<ItemsMixinClass> & T;
 
 export declare class ItemsMixinClass {
   /**