Added missing options #141, #143, #144 #142
Conversation
|
#143 Added the missing https://www.chartjs.org/docs/latest/configuration/elements.html#point-styles |
aripddev
changed the title
| /** | ||
| * The subtitle configuration is passed into the options.subtitle namespace. The global options for the chart subtitle is | ||
| * defined in Chart.defaults.global.subtitle. | ||
| */ |
There was a problem hiding this comment.
Looks like your Javadoc formatting is off?
There was a problem hiding this comment.
I use Ctrl+Shift+F to format the code and javadoc, but it changes the format completely. So, I made it manually :) Is there a better way to do it?
There was a problem hiding this comment.
Yeah you did the right thing not reformatting the whole file i think fixing it by hand is fine.
There was a problem hiding this comment.
I use Ctrl+Shift+F to format the code and javadoc, but it changes the format completely. So, I made it manually :) Is there a better way to do it?
Have a look at our CONTRIBUTING.md. Once setup correctly your IntelliJ does everything for you.
|
Thank you for the PR and the issues :) LGTM, however codestyle is broken. I will do a quick reformat. |
Fixes #141
Fixes #143
Fixes #144