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

Features to Enable Tornado Chart #140

Closed
wants to merge 22 commits into from

Conversation

clubajax
Copy link
Contributor

@clubajax clubajax commented Aug 11, 2017

We have a requirement to build a tornado chart. PPT has the ability to do this, but it requires customizing a typical bar chart series. This PR additionally has some general features.

  • catAxisLabelPos: By default the labels on the tornado chart will be on the 0 axis line. This allows for low (left) and high (right)
  • valAxisLineShow: Setting to false will hide the axis line (defaults true)
  • axisLineColor: color of axis lines
  • gridLineColor: color of grid lines (no features were added to handle line styles)
  • invertedColors: This is a list of colors, much like chartColors. These are applied if the value is less than zero. PPT defaults to white.
  • valueBarColors: Forces chartColors on multi-data-series. This is kind of a special switch that was specific to tornado charts. Not sure if it has any other use cases.

tornado-chart

@gitbrent gitbrent self-requested a review August 15, 2017 04:42
@gitbrent gitbrent self-assigned this Aug 15, 2017
gitbrent added a commit that referenced this pull request Aug 17, 2017
@gitbrent gitbrent mentioned this pull request Aug 17, 2017
@gitbrent
Copy link
Owner

Issue #151 added this code.

Thanks @clubajax !

@gitbrent gitbrent closed this Aug 17, 2017
@clubajax
Copy link
Contributor Author

@gitbrent, your welcome, but I don't see the changes in master?

@clubajax clubajax deleted the grid-formatting branch September 23, 2017 12:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants