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 prop to Bar to animate only when progress is increasing #43

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

Conversation

dlajarretie
Copy link

I needed to reset the bar progress after it was complete, several times in a row, but the animation made it look very confusing (it was going back and forth).

So I added this simple prop to animate the progress change only if the new value is greater than the previous one.

Not sure about the prop naming, or if it can be useful to others, but anyway, here you go.

dlajarretie and others added 23 commits February 9, 2017 11:35
* Add `strokeCap` prop for `Progress.Circle`

Changes: Allows for further customization of the `Progress.Circle` by adding a `strokeCap` prop, similar to the `Progress.CircleSnail` component.

* Add prop to Arc to allow for custom strokeCap

* Update project readme
* Add strokeCap prop to CircleSnail

* Update CircleSnail README
…oblador#65)

* fix strokeCap prop not being passed to foreground arc from Circle component

The default prop was being used, and the circle progress bar was always being
rendered with 'butt' type strokeCap

* add circlePercent and rotationOffset parameters to Circle.js

* Revert "add circlePercent and rotationOffset parameters to Circle.js"

This reverts commit b209640.
* Add native driver support and smooth bar animation

* update readme

* revert changes on circles and only change bar

* add animationOptions prop

* update readme to include animationOptions

* remove whitespace

* fix typo
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.

9 participants