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

[boxplot] add support for boxplot container events #80

Merged
merged 3 commits into from
Nov 29, 2017
Merged

[boxplot] add support for boxplot container events #80

merged 3 commits into from
Nov 29, 2017

Conversation

williaster
Copy link
Owner

@williaster williaster commented Nov 21, 2017 •
edited
Loading

This PR adds the following

🏆 Enhancements

  • Exposes the following props on <BoxplotSeries /> to enable more customization: containerProps, boxProps, outlierProps, minProps, maxProps, medianProps
  • Adds the ability to set mouse events on the boxplot container or on its component parts (whiskers, etc)
  • Consolidates some of examples for [demo][boxplot]:
    boxplot-violin-examples

TODO

@williaster williaster added this to the v0.0.49 milestone Nov 21, 2017
@coveralls
Copy link

coveralls commented Nov 21, 2017 •
edited
Loading

Coverage Status

Coverage decreased (-0.4%) to 83.44% when pulling 6aa5d09 on chris--boxplot-fixes into f5d4f42 on master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.7%) to 84.522% when pulling 8f7654a on chris--boxplot-fixes into f5d4f42 on master.

1 similar comment
@coveralls
Copy link

coveralls commented Nov 28, 2017 •
edited
Loading

Coverage Status

Coverage increased (+0.7%) to 84.522% when pulling 8f7654a on chris--boxplot-fixes into f5d4f42 on master.

@coveralls
Copy link

coveralls commented Nov 29, 2017 •
edited
Loading

Coverage Status

Coverage increased (+0.7%) to 84.522% when pulling 53c58fd on chris--boxplot-fixes into f5d4f42 on master.

@williaster williaster merged commit 2151811 into master Nov 29, 2017
@williaster williaster deleted the chris--boxplot-fixes branch November 29, 2017 20:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
v0.0.49
Development

Successfully merging this pull request may close these issues.
2 participants
@williaster @coveralls