retejs / context-menu-plugin

https://retejs.org
MIT License
11 stars 43 forks source link

Settings "delay" don't working. #70

Closed ovyavor closed 1 year ago

ovyavor commented 1 year ago

Hi. "Delay" does't work correctly in last stable version. I need to close context menu by click not only area of editor. It was a good idea to close by delay. How can I fixed it?

Ni55aN commented 1 year ago

How to reproduce it? If I specify this option in https://codepen.io/Ni55aN/pen/xzgQYq, it works correctly

ovyavor commented 1 year ago

You used the old version of plugin: https://cdn.jsdelivr.net/npm/rete-context-menu-plugin@0.6.0/build/context-menu-plugin.min.js This example with 1.0.0.alpha9 https://codesandbox.io/s/nifty-breeze-97vjku?file=/src/index.js

Ni55aN commented 1 year ago

this version isn't stable and was developed for Vue 3 compatibility. I would recommend using v2 (where the issue of compatibility is solved in other way), or I could accept the PR with the fix

ovyavor commented 1 year ago

Ok, thanks. I'll take v2.

rete-js[bot] commented 1 year ago

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 10 days.