PAN loop with maxBounds

[C-Moreira]

Hello.

I have a PAN loop when using maxBounds on map.
If I set maxBounds and zoom all the way to see the full map I get a PAN loop.
This happens for v0.6.4 and not for v0.5.1
To check the issue in v0.6 go to http://jsfiddle.net/8QHFe/642/
To see the same map in v0.5 without issue go to http://jsfiddle.net/8QHFe/644/

I have a fix changing function panInsideBounds so that if the view is larger than the map the map gets centered.

The code is:

panInsideBounds: function (bounds) {
    bounds = L.latLngBounds(bounds);

    var viewBounds = this.getPixelBounds(),
        viewSw = viewBounds.getBottomLeft(),
        viewNe = viewBounds.getTopRight(),
        sw = this.project(bounds.getSouthWest()),
        ne = this.project(bounds.getNorthEast()),
        viewCenter = L.bounds(viewSw, viewNe).getCenter(),
        boundsCenter = L.bounds(sw, ne).getCenter(),
        dx = 0,
        dy = 0;

    if (viewSw.y-viewNe.y > sw.y-ne.y) { // center vertical
        dy = boundsCenter.y-viewCenter.y;
    } else if (viewNe.y < ne.y) { // north
        dy = Math.ceil(ne.y - viewNe.y);
    } else if (viewSw.y > sw.y) { // south
        dy = Math.floor(sw.y - viewSw.y);
    }
    if (viewNe.x-viewSw.x > ne.x-sw.x) { // center horizontal
        dx = boundsCenter.x-viewCenter.x;
    } else if (viewNe.x > ne.x) { // east
        dx = Math.floor(ne.x - viewNe.x);
    } else if (viewSw.x < sw.x) { // west
        dx = Math.ceil(sw.x - viewSw.x);
    }

    if (dx || dy) {
        return this.panBy([dx, dy]);
    }

    return this;
},

Don't know if this is the best way to go.
Could some developer check this?
Thanks

[mourner]

I'll check this out, thanks. I think there was a similar report some time ago, so it may be a duplicate issue.
The best way to go about this is not to set minZoom on the map — this way it will be determined by the maxBounds and won't allow you to zoom all the way out.

[C-Moreira]

I checked fix for issue #1895 (#1895) as suggested by b_b in IRC, but it keeps the same result so I added an extra fix to getMinZoom like this:

getMinZoom: function () {
    var z1 = this._layersMinZoom === undefined ? 0 : this._layersMinZoom,
        z2 = this._boundsMinZoom === undefined ? 0 : this._boundsMinZoom,
        z3 = Math.max(z1, z2);
    return this.options.minZoom === undefined ? z3 : Math.max(z3, this.options.minZoom);
},

The problem is that with this fix I can't see my whole map extension. The zoom is too high. I suppose I would have to recalculate my map extension so it selects the next zoom level.