Leaflet: Allow changing the duration of zoom animations

[IvanSanchez]

Right now there is a duration option for pan animations, and for zoom/pan animations (fitbounds, setview). It would be nice to have a duration option also for the zoom animation, e.g.:

map.zoomIn(1, { duration: 0.5 })

The duration of the zoom animation is defined in two places: in the CSS definition of .leaflet-zoom-animated, and in the private _animateZoom method of L.Map.

The idea would be to get the duration out of the CSS, and define it dynamically in _animateZoom via this._mapPane.style[DomUtil.TRANSITION] = '...', along for the timeout for _onZoomTransitionEnd.

[hd719]

Hello can I work on this?

[perliedman]

@hd719 you're very welcome to do so, as far as I know no one is currently working on this.

[hd719]

Hey @perliedman I forked over the repo and I followed the instructions in the CONTRIBUTING.md file. When setting up the build system I ran both the commands npm install -g jake and npm install, but now how do I see the changes in the development environment? Do I have to run a specific command?

-Thank you.

[perliedman]

@hd719 if you run npm run watch, the build system will continously monitor the source for changes and rebuild.

[hd719]

Hey @perliedman I am having some trouble starting this I found the _animateZoom function in Map.js and having trouble moving on. Any advice?

-Thank you!

[shadeland]

Is this still open? I'm going to look into it

[perliedman]

@shadeland AFAIK there hasn't been anything done in this area.

It looks like I failed to respond to @hd719's question, but to be honest I'm not too familiar with this part of the code myself, so I will not be able to help with specifics on this issue.