Enhancement Request: onLocationFound about leaflet-locatecontrol

[domoritz]

Since I don't know about your requirements, why don't you add it and send a pull request which we then can discuss?

[wraybowling]

I second this request. I just ran into a need for being able to parse the data that results from the location found event. In my particular case, I'm going to push the resulting lat/long through the mapbox geocoder so i can display country/province/city.

Since Locate is being created with L.Control.extend() then I guess we should see how things like the geocoder enable the .on() function so events like 'found' fire events.

[domoritz]

@wraybowling in your case, you can just subscribe to the location found even on the map. It sounds like to don't want to change the logic of the locate control.

[wraybowling]

Could you provide an example? I thought I understood what you meant, but then I broke my fiddle. http://jsfiddle.net/rgbk/JgmE3/11/

[gmccrackin]

I'm not sure if you figured this out or not. So I will try to help if I can.

You had:

var map = L.mapbox.map('map', mapboxString)
          .addControl(
                L.mapbox.geocoderControl(mapboxString, {})
                .on('select',displayResults)
                .on('found',displayResults)
            )
            .addControl(
                L.control.locate().on('found', displayResults)
            )
;

I believe you want:

var map = L.mapbox.map('map', mapboxString)
          .addControl(
                L.mapbox.geocoderControl(mapboxString, {})
                .on('select',displayResults)
                .on('found',displayResults)
            )
            .addControl(
                L.control.locate()
            ).on('locationfound', displayResults)
;

This is because the event fires on the map object not the locate control. Hopefully this was helpful.

[domoritz]

Thanks @gmccrackin. I'm not sure what the right thing is so I fired the events on the map. If that is not the standard, it might be worth changing.

[gmccrackin]

Yeah I think firing it on the map is the standard for this.