Creates duplicate callbacks with multiple map instances about ol-contextmenu HOT 6 CLOSED

On it. We are losing the right map reference.

from ol-contextmenu.

@jonataswalker,

Please see the working repro at here which opens multiple windows with the 'test callback' menu item.

Thanks

from ol-contextmenu.

Thank you @jonataswalker! I've tested the latest v5.2.1 and the issue is resolved. Thanks again for all your awesome work.

from ol-contextmenu.

I haven't been able reproduce in a repro and its possible this is an Angular scoping issue. If I'm able to reproduce the issue, I'll repost. Thank you.

from ol-contextmenu.

@phelanHayes , Checked your repo and got confused. For a better clarity, I am suggesting you to create a context menu for any of the map and verify the behavior. Hope my point is clear.

from ol-contextmenu.

Hi @basilmohammed, I'm not sure I understand your comment. In the repro example, there are two maps and two contextmenus, one for each map. They each have separate event listener functions. Is this what you mean? I'm glad to add anything additional that is needed.

For the repro example, the corresponding event listener fires twice when invoked from its parent map/contextmenu. Thanks.

from ol-contextmenu.