Add z option to all annotations#738
Conversation
|
Changing the function name, we have re-enabled code climate issue about cognitive complexity. 😟 |
|
@LeeLenaleee apologize, I have tried to solve CC issue about complexity (I think better to have an exception as was before) and your approval is dismissed. |
|
np |
kurkle
left a comment
There was a problem hiding this comment.
Somewhat unrelated comment: Would be nice to normalize the common properties somehow, rather documenting for each annotation type.
Are you thinking a common MD file to import in each page or a specific page with common options? |
|
Common MD file that gets imported would be better, would be more in line with chart.js itself and will make it more user frienndly because people will only go to the place they are using and prob won't see the common page |
OK! The content for the next PR is already decided! ;) |
Fix #175
This PR is adding new option to all annotations and their label subelements in order to enable the user to define which annotations should be drawn on top, with the same
drawTime.