Leaflet.MultiOptionsPolyline icon indicating copy to clipboard operation
Leaflet.MultiOptionsPolyline copied to clipboard

Published 20 hours ago • verified publisher icon hgoebl

Doesn't paint lines for the first and last points properly.

Open jktu2870 opened this issue 6 years ago • 2 comments

  • If the first few points have the same "fnContext" result (0 for example), they will be painted in the color of the first point which has a different result (1 for example).
  • The mainline ends with the color of a penultimate point (not last).

jktu2870 avatar Mar 02 '18 15:03 jktu2870

Hi @jktu2870 can you tell me please why you already closed this PR? I had a short view of the code and it looks good to me, though I haven't tested it.

hgoebl avatar Mar 04 '18 07:03 hgoebl

Hi @hgoebl It's nice to know you estimated my PR as good. I closed PR because I have doubt about performance since "fnContext" now cause twice per iteration. Also, I didn't test this solution for every case MultiOptionsPolyline can be used for, only for examples on the demo page and for my specific task. I'll reopen PR, if you have time to test it then it would be great.

jktu2870 avatar Mar 05 '18 12:03 jktu2870