Skip to content

Option to animate the drawing of the graph#62

Closed
davidgtonge wants to merge 5 commits into
morrisjs:masterfrom
davidgtonge:master
Closed

Option to animate the drawing of the graph#62
davidgtonge wants to merge 5 commits into
morrisjs:masterfrom
davidgtonge:master

Conversation

@davidgtonge
Copy link
Copy Markdown

Options added:

  • animate (boolean) - flag on whether to animate or not

  • duration (number) - duration of the animation

  • easing (string) - one of Raphael's supported easing options

    Methods changed:

  • drawSeries - added animation code

Method added:

  • seriesAverage - caculates the average y value for a set of coords

    NB diff fixed, minified js not modified

Dave Tonge added 5 commits May 25, 2012 10:07
Options added:
 - animate (boolean) - flag on whether to animate or not
 - duration (number) - duration of the animation
 - easing (string) - one of Raphael's supported easing options

 Methods changed:
 - drawSeries - added animation code

Method added:
 - seriesAverage - caculates the average y value for a set of coords

 NB minified js not modified
@iamjoshua
Copy link
Copy Markdown

6 months old... any chance?

@tkjaergaard
Copy link
Copy Markdown

+1

@jesper-bylund
Copy link
Copy Markdown

Did this happen or not? Is there any animation?

@oesmith
Copy link
Copy Markdown
Contributor

oesmith commented Jan 25, 2013

Like some other pull requests I've cleared out today, I'm not convinced this one's worth the test/maintenance overhead.

Closing.

@oesmith oesmith closed this Jan 25, 2013
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.

5 participants