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

Morris.Bar #98

Closed
tiraeth opened this issue Oct 26, 2012 · 2 comments · Fixed by #101
Closed

Morris.Bar #98

tiraeth opened this issue Oct 26, 2012 · 2 comments · Fixed by #101

Comments

@tiraeth
Copy link
Contributor

tiraeth commented Oct 26, 2012

Hello Olly,

Recently I have done Morris.Bar (unfortunately, only one serie per chart) that nicely draws a bar graph (bars can be drawn using gradients - two colors or two colors + middle color linear gradients). Unfortunately I've done it before pulling your latest updates (this means - moving Grid plotting to parent class and so on).

To allow the graph to support multiple series (e.g. as drawing bars one next to another) you will need to fix createBar and drawSeries functions.

Anyway, if you are willing to update my Morris.Bar to work nicely with your recent version, I would be glad. I really need this type of chart (so I made it) and I think many will find it useful :-)

https://gist.github.com/d15e9aad70d36cb8a50d

@oesmith
Copy link
Contributor

oesmith commented Oct 27, 2012

That's great, thanks!

I'm working on area charts (and beefing up the test suite) at the moment, which is why there's been so much refactoring recently. I'll try to get this integrated as soon as the area charts are done.

👍

@kwando
Copy link

kwando commented Oct 30, 2012

+1

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 a pull request may close this issue.

3 participants