@vaadin/context-menu 23.1.0-rc2 → 23.1.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@vaadin/context-menu",
-  "version": "23.1.0-rc2",
+  "version": "23.1.1",
   "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-rc2",
-    "@vaadin/item": "23.1.0-rc2",
-    "@vaadin/list-box": "23.1.0-rc2",
-    "@vaadin/lit-renderer": "23.1.0-rc2",
-    "@vaadin/vaadin-lumo-styles": "23.1.0-rc2",
-    "@vaadin/vaadin-material-styles": "23.1.0-rc2",
-    "@vaadin/vaadin-overlay": "23.1.0-rc2",
-    "@vaadin/vaadin-themable-mixin": "23.1.0-rc2"
+    "@vaadin/component-base": "^23.1.1",
+    "@vaadin/item": "^23.1.1",
+    "@vaadin/list-box": "^23.1.1",
+    "@vaadin/lit-renderer": "^23.1.1",
+    "@vaadin/vaadin-lumo-styles": "^23.1.1",
+    "@vaadin/vaadin-material-styles": "^23.1.1",
+    "@vaadin/vaadin-overlay": "^23.1.1",
+    "@vaadin/vaadin-themable-mixin": "^23.1.1"
   },
   "devDependencies": {
     "@esm-bundle/chai": "^4.3.4",
-    "@vaadin/polymer-legacy-adapter": "23.1.0-rc2",
+    "@vaadin/polymer-legacy-adapter": "^23.1.1",
     "@vaadin/testing-helpers": "^0.3.2",
     "sinon": "^13.0.2"
   },
-  "gitHead": "154c6782c42145fed21e443559fbe2d781ad3ec7"
+  "gitHead": "390458d6519433a2dd502cef90da48e84573a275"
 }

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.js

@@ -3,7 +3,6 @@
  * Copyright (c) 2016 - 2022 Vaadin Ltd.
  * This program is available under Apache License Version 2.0, available at https://vaadin.com/license/
  */
-import { flush } from '@polymer/polymer/lib/utils/flush.js';
 import { isTouch } from '@vaadin/component-base/src/browser-utils.js';
 import { Item } from '@vaadin/item/src/vaadin-item.js';
 import { ListBox } from '@vaadin/list-box/src/vaadin-list-box.js';
@@ -273,20 +272,18 @@ export const ItemsMixin = (superClass) =>
     /** @private */
     __initMenu(root, menu) {
       if (!root.firstElementChild) {
-        const is = this.constructor.is;
-        root.innerHTML = `
-        <vaadin-context-menu-list-box></vaadin-context-menu-list-box>
-        <${is} hidden></${is}>
-      `;
-        flush();
-        const listBox = root.querySelector('vaadin-context-menu-list-box');
+        const listBox = document.createElement('vaadin-context-menu-list-box');
+        root.appendChild(listBox);
+
         if (this._theme) {
           listBox.setAttribute('theme', this._theme);
         }
         listBox.classList.add('vaadin-menu-list-box');
         requestAnimationFrame(() => listBox.setAttribute('role', 'menu'));
 
-        const subMenu = root.querySelector(is);
+        const subMenu = document.createElement(this.constructor.is);
+        subMenu.setAttribute('hidden', '');
+        root.appendChild(subMenu);
         subMenu.$.overlay.modeless = true;
         subMenu.openOn = 'opensubmenu';