-
Notifications
You must be signed in to change notification settings - Fork 9
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
TypeError: Cannot read property 'style' of undefined #6
Comments
@ToHold - the author of this plugin obviously abandoned it. Which is why I find it interesting that Leaflet still keeps it on their site even though it's completely broken with the new leaflet. However, not to fear a colleague of mine and myself have been working the last several days getting it migrated over to leaflet 1.5.x .. We have a prelim version working. Once we're done we will fork this repo and create our own. to Provide a working copy of this (with more documentation!!! and more functionality). |
@ToHold fixed @xmikedanielsx migrating it to leaflet@1.x was about 20 minutes. I never had much push for that, and I somehow missed this ticket (I have a lot of public repositories). Please refrain from bitter messages like that, instead you can always fork the repo and make a PR. Excited to see the new functionality you were working on! |
@w8r -- please understand my comment was not meant in any means to be "bitter". I was really impressed with this plugin and truly do love the work you did. So much I recommended it for my work. I have been butchering it unfortunately in my code because I do not understand quite how to use it (from a source stand point). However I am pushing myself through it. Code is clean :) Until I got a hold of it. haha Things I needed I have managed to butcher together
Things I would love to have
Thank you again so much for your amazing work and contribution :) 👍 👍 👍 |
@xmikedanielsx 👍 /**
* Replace this method if you want to subclass moving marker
* @param {L.LatLng} pos
* @param {Object} style
* @return {L.Control.LineStringSelect.ControlMarker}
*/
movingMarkerFactory: function(pos, style) {
return new ControlMarker(pos, style);
},
/**
* Replace this method if you want to subclass endpoint marker
* @param {L.LatLng} pos
* @param {Object} style
* @param {Boolean} isEnd
* @return {L.Control.LineStringSelect.Endpoint}
*/
endpointFactory: function(pos, style, isEnd) {
return new Endpoint(pos, style);
}, styles for those are passed via control options, so that's quite straight-forward, too |
@w8r 👍 |
around here you can find how to get the meter marks of selection start and end |
as for Vue.js - I don't really know that framework, I can't help you there |
it's not really about vuejs -- this I know how to use very well. |
Hello,
Nice work!!! This is a really cool component !
But with Leaflet 1.3.1, i got an issue, can you help me ?
This is what my google console show me :
https://image.noelshack.com/fichiers/2018/20/4/1526566662-scene-1-2018-05-17-16-15-14.jpg
The text was updated successfully, but these errors were encountered: