jarek-foksa / xel

Xel - Widget toolkit for building native-like Electron and Web apps
https://xel-toolkit.org
Other
689 stars 59 forks source link

x-menu Quick click will cause the page card to die #111

Closed u10k closed 2 years ago

u10k commented 2 years ago

Bug Type: Component

Environment

Reproduction

Related Component

Reproduction Link

Docs

Steps to reproduce

Right -click quickly

image

jarek-foksa commented 2 years ago

Can you reproduce this bug with the latest version of Chrome? You seem to be using Chrome 89 which is very old.

u10k commented 2 years ago

Can you reproduce this bug with the latest version of Chrome? You seem to be using Chrome 89 which is very old.

I found nothing to do with chrome version, Affected by [x-contextmenu]

<x-menu id="x-menu">
   <x-menuitem>copy</x-menuitem>
</x-menu>
document.body).addEventListener('contextmenu', (e) => {
            e.preventDefault();
            const elMenu = document.getElementById('x-menu');
            elMenu.openAtPoint(e.clientX, e.clientY);
 });

if i remove x-contextmenu, is good

jarek-foksa commented 2 years ago

Please report this bug on https://bugs.chromium.org/p/chromium/, I can't do it as I'm unable to reproduce the crash. In the Chromium bug report you should include the 16 digit crash ID as shown on chrome://crashes

u10k commented 2 years ago

Please report this bug on https://bugs.chromium.org/p/chromium/, I can't do it as I'm unable to reproduce the crash. In the Chromium bug report you should include the 16 digit crash ID as shown on chrome://crashes

you can run this https://github.com/u10k/xelMenu

jarek-foksa commented 2 years ago

Please provide unobfuscated test case.

u10k commented 2 years ago

Please provide unobfuscated test case.

demo video

jarek-foksa commented 2 years ago

Thanks, I managed to reproduce the bug and I fixed it in Xel version 0.17.2.