Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[BUG] Tick documentation is missing autoSkipPadding #3490

Closed
bretjb opened this issue Oct 19, 2016 · 2 comments
Closed

[BUG] Tick documentation is missing autoSkipPadding #3490

bretjb opened this issue Oct 19, 2016 · 2 comments
Labels
help wanted type: bug type: documentation
Milestone
Version 2.6

Comments

@bretjb
Copy link
Contributor

bretjb commented Oct 19, 2016

Currently, there is an undocumented feature that will increase the padding between ticks when auto skip is enabled and it has been activated.

Current Behavior

autoSkipPadding is not included in the documentation, but it is in the tests and in source where it is used in calculating where ticks should be drawn on the canvas.

Expected Behavior & Possible Solution

The tick documentation should have something like the following in the Scales documentation, under 'Ticks'
autoSkipPadding: number - padding between ticks on the same axis when values have been skipped.

Environment

  • Chart.js version: 2.2.2

Notes

If this is accepted as a bug, I can submit a PR with a fix. :) Thanks
@etimberg
Copy link
Member

@bretjb I would be happy to look at a PR for this

@etimberg etimberg added this to the Version 2.5 milestone Oct 23, 2016
etimberg pushed a commit that referenced this issue Oct 25, 2016
@bretjb @etimberg
Fixes issue #3490: Tick documentation is missing autoSkipPadding
295c9db 
Adding a single line to the docs to document autoSkipPadding.
etimberg added a commit that referenced this issue Dec 30, 2016
@etimberg
Moved docs over to new folder structure. Removed old doc files except…
86d4b13 
… for the comparison table. Fixed #3547, #3490, and #3118  while at it.
@etimberg etimberg mentioned this issue Dec 30, 2016
Merged
etimberg added a commit that referenced this issue Jan 8, 2017
@etimberg
Moved docs over to new folder structure. Removed old doc files except…
0bc3410 
… for the comparison table. Fixed #3547, #3490, and #3118  while at it.
etimberg added a commit that referenced this issue Jan 8, 2017
@etimberg
Moved docs over to new folder structure. Removed old doc files. Fixed #…
3815036 
…3547, #3490, and #3118  while at it.
@etimberg etimberg modified the milestones: Version 2.6, Version 2.5 Jan 15, 2017
etimberg added a commit that referenced this issue Feb 18, 2017
@etimberg
Moved docs over to new folder structure. Removed old doc files. Fixed #…
63b2004 
…3547, #3490, and #3118  while at it.
@etimberg
Copy link
Member

Fixed in #3751

exwm pushed a commit to exwm/Chart.js that referenced this issue Apr 30, 2021
@bretjb @etimberg
Fixes issue chartjs#3490: Tick documentation is missing autoSkipPadding
59fcd15 
Adding a single line to the docs to document autoSkipPadding.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
help wanted type: bug type: documentation
Projects
None yet
Milestone
Version 2.6
Development

No branches or pull requests
2 participants
@bretjb @etimberg