Huge CPU Usage when zoomed in about leaflet.curve HOT 6 CLOSED

Hello!
Can you provide more details about your environment.

• What hardware (model, CPU, RAM), operating system, and browser are you seeing the issue on?
• To what level are you zoomed in when you start seeing the issue?
• Is the issue limited to curves with animations and/or dashes only or does it happen with non-animated curves with solid lines?
• Are you able to quantify the CPU usage (it jumps from __% usage to __% usage)?

from leaflet.curve.

@elfalem Close this

from leaflet.curve.

@elfalem Close this

This is still an issue I wonder why it was closed, if you zoom in to an animated line close enough, it lags a whole lot :(
Can test it with the demo project: http://elfalem.github.io/Leaflet.curve/

from leaflet.curve.

After some more testing, the lag ONLY occurs to dashed lines, whether animation is turned on or off does not affect CPU usage.
It seems to be related to the way the dashed lines are generated.

from leaflet.curve.

After some more testing, the lag stems from the fact that if you zoom into the map, the distance between point A and point B gets larger in terms of pixels, it then has to create more little dashed lines to cover the increased distance between point A and point B (since the length of each dash is constant).

Tested it with the code in this PR: #54
Instead of creating more little dashed lines, the PR keeps the number of dashed lines constant and makes each dashed line longer instead to cover the increased distance between point A and point B.

It will look like this now when zoomed in:

The PR fixes the lag, but makes the dashed lines look stretched out and weird. The lag is actually still there IF you create the line while you are already zoomed in.

from leaflet.curve.

Created a PR #57 to potentially fix this.

from leaflet.curve.