Skip to content

Make major ticks optional and off by default#4723

Merged
etimberg merged 1 commit intochartjs:masterfrom
benmccann:majorTicksOptional
Sep 8, 2017
Merged

Make major ticks optional and off by default#4723
etimberg merged 1 commit intochartjs:masterfrom
benmccann:majorTicksOptional

Conversation

@benmccann
Copy link
Contributor

Leave the major tick functionality off by default. It still has some polishing left to do and many people don't expect the change

@benmccann benmccann mentioned this pull request Sep 5, 2017
4 tasks
@etimberg etimberg requested a review from simonbrunel September 6, 2017 22:01
@simonbrunel
Copy link
Member

Based on this issue, I think we should also disable major alignment if ticks.major.enabled: false.

@benmccann
Copy link
Contributor Author

Ok, I agree at least in the short term because the alignment isn't working very well currently. On that issue, he said scale.ticks.source: 'data' or 'labels' gave him the behavior he wanted. He still has an issue with the autoSkipper, but that's orthognal. I have the same issue as well. I'd like to try addressing both of these issues soon.

@benmccann
Copy link
Contributor Author

I've updated the PR to make that change as well

@etimberg etimberg merged commit 427aa99 into chartjs:master Sep 8, 2017
yofreke pushed a commit to yofreke/Chart.js that referenced this pull request Dec 30, 2017
exwm pushed a commit to exwm/Chart.js that referenced this pull request Apr 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants