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.

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

Alberto acevedo patch 1 #28

Open
wants to merge 19 commits into
base: master
Choose a base branch
from

Conversation

alberto-acevedo
Copy link

The onRemove is failing to remove the _onMouseMove event because the context is missing. In Leaflet 1.8.0 it logs the following warning: listener not found. The fix is to include the context (this) in the onRemove function.

danwild and others added 19 commits May 25, 2016 11:49
This allows you to use the library to show other coordinates than lat/lng.
It gives you the lat/lng in one function to be able to do translations.
Add option to not wrap longitude values
Add formatting function that formats both lat and long at the same time
The onRemove is failing to remove the _onMouseMove event because the context is missing. In Leaflet 1.8.0 it logs the following warning: listener not found.
The fix is to include the context (this) in the onRemove function.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants