Skip to content

Adding optional animation for drawing the lines#66

Open
joaortk wants to merge 2 commits intoaeyrium:masterfrom
joaortk:master
Open

Adding optional animation for drawing the lines#66
joaortk wants to merge 2 commits intoaeyrium:masterfrom
joaortk:master

Conversation

@joaortk
Copy link

@joaortk joaortk commented Dec 13, 2019

This is a simple addition, allowing the user to enable animation on the line drawing.
animation

);

if(widget.animateAppear) {
var controller = AnimationController(
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can you change to final instead of var here? controller won't be modified

@diegoveloper
Copy link
Collaborator

Thanks for the contribution !! I added a small comment in your code

@diegoveloper
Copy link
Collaborator

One question I have, did you try with a big chart data? like weekly scale? Could you add a GIF about that case?

@diegoveloper
Copy link
Collaborator

Hi @joaortk do you have any news about the feedback I gave you ?

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.

2 participants