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

Prints data value labels for all series. #57

Closed
wants to merge 1 commit into from

Conversation

lbrito1
Copy link

@lbrito1 lbrito1 commented Sep 8, 2017

Hitherto only the first serie would have data value text labels printed.

More info: #56

Hitherto only the first serie would have data value text labels printed.
@lbrito1
Copy link
Author

lbrito1 commented Sep 8, 2017

FYI I'm not sure how this impacts the rest of the gem.

@coveralls
Copy link

coveralls commented Sep 8, 2017

Coverage Status

Coverage decreased (-18.8%) to 81.176% when pulling f1eb130 on lbrito1:master into c8fa02c on Fullscreen:master.

@lbrito1
Copy link
Author

lbrito1 commented Sep 8, 2017

Found out that I had to use labels: [true, true] option. Ideally this should be in the documentation somewhere. If it is, sorry, I missed it!

@lbrito1 lbrito1 closed this Sep 8, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants