New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add includeBounds option for cartesian ticks #9020
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code looks good, would be good to have a test. Types need to be updated as well
Good calls, it does not actually even work currently :) |
Is this ready to go? |
It could be separated to min/max, as it can be scriptable and in that case it would be more versatile. But there is no order for such a thing, so it's ready as far as I'm concerned. |
Actually, I'm going to add a fix for #9025 while at it. |
Is there more to do on this, or is it good to merge? |
This behavior is not suited for all use cases, in particular with the
chartjs-plugin-zoom
and the fact that when zoomed, the min / max values can vary greatly in precision compared to other ticks.Fixes chartjs/chartjs-plugin-zoom#342