DACircularProgress icon indicating copy to clipboard operation
DACircularProgress copied to clipboard

Progress jumps in 2.3

Open Igor-Palaguta opened this issue 9 years ago • 4 comments

Animation jumps if animation is not completed yet and we set new progress value. Even incorrect value can be set, like in mentioned above example: self.progressView.setProgress(1.0, animated: true) //....and when I call during animation progress self.progressView.setProgress(0.5, animated: false) last call initiates remove of animation. And it calls animationDidStop delegate method asynchronously, even after proper progress is set. So after 0.5 is set, 1.0 will be set in animationDidStop in a moment

Version 2.2 without animation delegate works fine

Igor-Palaguta avatar Jun 09 '15 14:06 Igor-Palaguta

i can confirm that

peterpaulis avatar Jul 09 '15 11:07 peterpaulis

Look like this change broke that:

a7f53525b8250e8a416e055c2f29e9b622f4a8f1

The issue is that this bit of code was moved inside the else and not executed in the animated block:

  •    self.circularProgressLayer.progress = pinnedProgress;
    

Moving it out of the else again fixes the issue.

woodsnick avatar Aug 11 '15 04:08 woodsnick

+1

AcidicSkittles avatar Aug 31 '15 06:08 AcidicSkittles

hi guys! FYI hi tried to make that change but didn't work, but I manage to solve it, look the PR: https://github.com/danielamitay/DACircularProgress/pull/67

hope it helps.

BTW, sometimes animations don't work at all, but I don't know exactly when and so why, because any of the progress views in the app work, so maybe is related somehow w/ the network layer... but I would say it started to happen when updated to >= 2.3

iamyellow avatar Oct 10 '15 09:10 iamyellow