chartjs / chartjs-plugin-zoom

Zoom and pan plugin for Chart.js
MIT License
596 stars 327 forks source link

Fix handling at canvas edge #898

Closed joshkel closed 1 day ago

joshkel commented 2 days ago

This is a possible fix for #897. Further testing may be needed.

joshkel commented 1 day ago

@kurkle I have a test in my local branch at https://github.com/joshkel/chartjs-plugin-zoom/tree/local-2.1.0. It depends on #896 being merged. I can wait for that PR, or, if you'd rather have one PR that combines these fixes, I can do that.

kurkle commented 1 day ago

@joshkel #896 is merged

joshkel commented 1 day ago

@kurkle Thanks. I've added tests.

kurkle commented 1 day ago

Thanks @joshkel

I will add at least one more thing tomorrow and publish a release.