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

added gpx track length into popup #25

Merged
merged 3 commits into from
Oct 24, 2013
Merged

Conversation

stefanocudini
Copy link
Contributor

i mak etwo utility methods, _humanLen(), _polylineLen() and insert value into popup

I think that the know the length of a gpx track is very useful more of the name and description

@brunob
Copy link
Collaborator

brunob commented Sep 21, 2013

Hi @stefanocudini and thanks for the patch. This one need to be fixed cause tested it and it throws this error in gpx.html example :

TypeError: line.getLatLngs is not a function vector/GPX.js Line 39

@brunob
Copy link
Collaborator

brunob commented Oct 24, 2013

Hi again @stefanocudini, could you update your patch in order to fix the bug i was talking about in my previous post ?

@stefanocudini
Copy link
Contributor Author

Hi @brunob I think that this resolve the problem:
stefanocudini@77d002c

but I'm not sure, it's been a long time since I made this change.
However the GPX demo now work!

brunob added a commit that referenced this pull request Oct 24, 2013
added gpx track length into popup
@brunob brunob merged commit 42d491d into shramov:master Oct 24, 2013
@brunob
Copy link
Collaborator

brunob commented Oct 24, 2013

Merged, thanks for the patch and the update @stefanocudini :)

@avoulvou
Copy link

Hi,

I use kml plugin , all info updated without problem and I want to thank you for this plugin.
I want to display a text next to icon, the text its the name - title in kml
Something like that : http://leaflet.github.io/Leaflet.label/ Any idea - hint please
It can displayed and as popup , I just want to display the name or title

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.

3 participants