Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Inherit from L.Evented instead of L.Mixin.Events for custom events #47

Closed
SBats opened this issue Aug 3, 2017 · 4 comments
Closed

Inherit from L.Evented instead of L.Mixin.Events for custom events #47

SBats opened this issue Aug 3, 2017 · 4 comments
Labels
bug

Comments

@SBats
Copy link
Contributor

SBats commented Aug 3, 2017

As stated in console L.Mixin.Events is now deprecated and will be removed and any custom event should instead inherit from L.Evented.
@SBats SBats added the bug label Aug 3, 2017
@ldoloscan
Copy link

The same here,
any workaround to get rid of this WARNING ?

@SBats
Copy link
Contributor Author

SBats commented Aug 4, 2017

The plugin needs to be updated. I don't think it's a lot of work if you want to try a PR :).
Otherwise, I'll try to give it a look but I have no idea when I'll find time for this.

@ppete2
Copy link

ppete2 commented Sep 27, 2017

Bug here too with Leaflet v1.2

@SBats
Copy link
Contributor Author

SBats commented Nov 18, 2017

Fixed with the release 1.2.0! #51

@SBats SBats closed this as completed Nov 18, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@SBats @ppete2 @ldoloscan