search

michitaro / vue-menu

Menu/Contextmenu Component for vue2
https://michitaro.github.io/vue-menu/
MIT License
263 stars 26 forks source link

compatible with nested contextmenu #10

Closed frankxin closed 6 years ago

frankxin commented 6 years ago

In a nested context menu case, one click will show two menus because of not stopping propagation. The showcase detail is in sample3.

michitaro commented 6 years ago

Thank you very much for your PR. Your example for nested context menu is very useful, but this is enough for nested context menu.

If the library had .prevent.stop, consumers have no way to catch the original event, so putting .prevent.stop should be done by consumer.

Can I merge your example for nested context menu?

frankxin commented 6 years ago

Feel free to merge the example ~ You are totally right about it! Thank you.

michitaro commented 6 years ago

Thanks!