Adding a circlemarker with L_PREFER_CANVAS enabled causes an infinite loop of moveend firings.

[snkashis]

Try out http://bl.ocks.org/snkashis/5666125 , and take a look at the console.
Adding a circlemarker fires 'moveend' on the map in an infinite loop.

This issue is present in 0.5.1 or 0.6-dev.

Removing the this.fire('moveend');
in https://github.com/Leaflet/Leaflet/blob/master/src/layer/vector/canvas/Path.Canvas.js#L56

fixes this up temporarily but obviously we need to dig a bit deeper here.

Bisecting indicates we should be looking at bd8d5ff

[chrisincambo]

This issue isn't fixed in 7.2 or 7.3. The moveend event is still on an infinite loop when circle markers are rendered and L_PREFER_CANVAS has been set to true.

[perliedman]

Can you provide a jsfiddle or other online example?

[chrisincambo]

Sure here's an example: https://gist.github.com/chrisincambo/6ef2638b6d3eaa018ea2

[mattiasb]

Converted the gist above into a fiddle (using 0.7.3). I don't see any problem.

EDIT: sorry, I see the problem in the gist but can't seem to produce a fiddle of it reproducing it. Sorry for the noise.

[perliedman]

@chrisincambo that gist still includes Leaflet 0.5.1?

[chrisincambo]

Oops, I just spotted this too. I'm going to have to do a little more work
tracking this down, using the GIST code it also works for me now I've
updated the version, but in our app we're still seeing the infinite loop.
There's a pretty complex lead up to this call in our app, so we're just
just going to slice things out until we can isolate the the exact
configuration/data that causes this, then I will report back. Thx!

On Fri, Oct 3, 2014 at 4:22 PM, Per Liedman notifications@github.com
wrote:

@chrisincambo https://github.com/chrisincambo that gist still includes
Leaflet 0.5.1?

—
Reply to this email directly or view it on GitHub
#1712 (comment).