Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

L.geoJson() layer position gets reset on pan #10

Open
monkut opened this issue Apr 24, 2013 · 2 comments
Open

L.geoJson() layer position gets reset on pan #10

monkut opened this issue Apr 24, 2013 · 2 comments

Comments

@monkut
Copy link

monkut commented Apr 24, 2013

I just plugged in the debug demo code below into a minimal map where L.geoJson() layers already exist, to try out rlayer.

On pan both existing L.geoJson() layer's position gets reset to the center.
(this shouldn't be happening.)

Then tried to replace L.geoJson() with R.GeoJSON() and got a leaflet-side id error.

using Leaflet-Leaflet-v0.5.1-0-gc1d410f.zip.

var adelaide = new L.LatLng(-34.93027490891421, 138.603875041008);
var points = [];
map.on('click', function(e) {
points.push(e.latlng);

if(points.length == 4) {
    var p = new R.Polygon(points);
    map.addLayer(p);
    p.hover(function() {

    },
    function() {
        p.animate({opacity: 0}, 1000, function() { map.removeLayer(p); });

    });

    points = [];
}


var b = new R.BezierAnim([adelaide , e.latlng], {}, function() {
    var p = new R.Pulse(
            e.latlng, 
            6,
            {'stroke': '#2478ad', 'fill': '#30a3ec'}, 
            {'stroke': '#30a3ec', 'stroke-width': 3});

    map.addLayer(p);
    setTimeout(function() {
        map.removeLayer(b).removeLayer(p);
    }, 3000);
});

map.addLayer(b);

});

@ghost
Copy link

ghost commented Jan 11, 2014

Ah ! I am facing the same problem :( I am using Pulse too. Any solutions ?

@pvinton
Copy link

pvinton commented Jan 11, 2014

I submitted a pull request a couple of months ago that I think may solve this problem: #14

The pull request has not been accepted, so you'll have to manually tweak your code to test it out.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants