Feature request: Ability to mofidy homeCoordinates after init about leaflet.zoomhome

[m1dst]

@answerquest I've added the code and done a brief test. Seems OK to me, give it a try and let me know if it doesn't do what you expected.

[answerquest]

Hi, this feature is very much needed. As an alternative to passing co-ordinates, how about passing bounds? Then I can pass in a layer's getBounds() whenever the layer has changes.

[m1dst]

I've added a pull request for this issue. #11

[answerquest]

@m1dst this is great! Request: can you add in a setHomeBounds method too? That would take as argument a LatLngBounds array, like from a layer.getBounds().

Possible method to use : getBoundsZoom http://leafletjs.com/reference-1.3.0.html#map-getboundszoom

[answerquest]

@m1dst a small correction is needed, once that is done it works well! I added this code to the demo page to test:

var markerLayer = new L.geoJson(null);
var marker1 = L.marker([0,0]).addTo(markerLayer);
var marker2 = L.marker([20,20]).addTo(markerLayer);
markerLayer.addTo(map);
zoomHome.setHomeBounds( markerLayer.getBounds() );

Oh, I had to change the included script from leaflet.zoomhome.min.js to leaflet.zoomhome.js ; minified version didn't have the setHomeBounds method added yet.

[m1dst]

@answerquest Fixed in a subsequent update. I had a variable called map which is why it worked for me. Stupid typo.

[m1dst]

Thanks for all your feedback. I have closed the original PR and moved the new code to a new branch targeting just this issue rather than pointing at my main branch. I raised a new PR #12 where I have also minified the file.

[torfsen]

This has been closed via #12. Thanks, @m1dst!