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.

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

Added PLUGIN ID because missing #173

Merged
merged 2 commits into from Aug 14, 2019
Merged

Added PLUGIN ID because missing #173

merged 2 commits into from Aug 14, 2019

Conversation

stockiNail
Copy link
Collaborator

Added PLUGIN ID 'annotation" in order to register into Chart.js as required like best practise.
In this way you could disable in some chart instances, if you want

Added PLUGIN ID 'annotation" in order to register into Chart.js as required like best practise.
In this way you could disable in some chart instances, if you want
remove empty row
@benmccann benmccann merged commit 27f90f8 into chartjs:master Aug 14, 2019
osro added a commit to osro/chartjs-plugin-annotation that referenced this pull request Jan 28, 2020
* commit '7037eea71314613a8af83d2298241841cbc34c4f':
  Fix undefined reference
  "preserveComments" not longer available on gulp-uglify (chartjs#177)
  Added PLUGIN ID because missing (chartjs#173)
  allow multiple lines of text in the line label (chartjs#129)
  Chart.js should be a peerDependency
  Fix getting chart borders for box size setting (chartjs#152)
  Fix a number of lint errors in the samples
  Add missing .htmllintrc
  Lint sample html (chartjs#150)
  Pass the index of value datapoint to getPixelValue (chartjs#109)
  move configure step to draw time, to support render call (chartjs#116)
  Read plugin options from plugins.annotation (chartjs#130)
  Add Travis config
  Add gulp lint task (chartjs#144)
  Fix Dropbox url

# Conflicts:
#	README.md
#	package.json
@kurkle kurkle added the bug label Oct 23, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants