Don't clip every single point

hansmaad / chartist-plugin-zoom

Zoom Plugin for Chartist.js

Do What The F*ck You Want To Public License

17 stars 15 forks source link

Don't clip every single point #18

Closed SleepProgger closed 5 years ago

SleepProgger commented 5 years ago

This PR changes the clipping behavior to only require one 'clip-path' attribute per series plus one for each line, bar and area. Previously a 'clip-path' attribute was set for each point, which for graphs with many points can lead to performance problems (Although not really dramatic ones TBH). Anyhow this shouldn't harm and increase the performance a bit.

hansmaad commented 5 years ago

@SleepProgger Thank you for your contributions! I hope I'll have some time to look at it on the weekend :)

SleepProgger commented 5 years ago

No worries. I mostly need this features for myself. So it wouldn't be thaat dramatic if not ;)