zoom to integer levels

[hyperknot]

Round target zoom levels for zoom controls, like it's done at DoubleClickZoom already:

Leaflet/src/map/handler/Map.DoubleClickZoom.js

Line 21 in 161172b

zoom = e.originalEvent.shiftKey ? Math.ceil(oldZoom) - 1 : Math.floor(oldZoom) + 1;

[IvanSanchez]

No. See #3523 - with fractional zoom, it's not about rounding anymore, it's about snapping to a number defined as a map option (default 1).

@hyperknot I would appreciate if you could review that PR, though.

[hyperknot]

It'll take some time to read #3523 and the linked issues in that one, but as of now, there is inconsistent behaviour between DoubleClickZoom (13.5 -> 14 -> 15) and zoom controls (13.5 -> 14.5 -> 15.5).

[IvanSanchez]

@hyperknot I see your point, but I'd rather work on #3523 instead of patching this up and creating a merge conflict :-/

[hyperknot]

@IvanSanchez I agree, I'll review that one.