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

Add support for stacked area in timeline graphs #1

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

jcmuller
Copy link

@jcmuller jcmuller commented Jun 1, 2012

No description provided.

jcmuller added 4 commits May 31, 2012 17:05
…values on y Axis, since Highcharts handles that rather well
This was done so that users can have a widget with a, for example,
stacked area chart, for which you'd set the following options:

    widget "Users", {
      :title => "Some Title",
      :type => :timeline,
      :plot_style => :area,
      :plot_options => { 'area' => { 'stacking' => 'normal' } },
      :gauges => [ :my_gauge ]
    }

Another nice option is 'stacking' => 'percent'
@midu
Copy link

midu commented Jun 4, 2012

the only thing that bothers me is that the tests are not passing (on my machine). It mainly looks like typing errors... Do they pass on your machine?

@jcmuller
Copy link
Author

jcmuller commented Jun 4, 2012

Master isn't passing either, with the same failures...

@midu
Copy link

midu commented Jun 5, 2012

I know, the thing is that travis says that the original repo is passing. I pulled it and ran rake to see it they passed but they didn't (pretty much the same errors).

I was more wondering if we're setting the environment properly.

other than that, feel free to move this to qa

@rossta
Copy link

rossta commented Jun 8, 2012

Let's get a pull request up to the main project!

@jcmuller
Copy link
Author

jcmuller commented Jun 8, 2012

Done. Waitin'

Juan C. Müller

On Friday, June 8, 2012 at 11:18 AM, Ross Kaffenberger wrote:

Let's get a pull request up to the main project!


Reply to this email directly or view it on GitHub:
#1 (comment)

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.

3 participants