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

'pulse_startFromScratch' fix for new Android versions #29

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

Conversation

m-kubis
Copy link

@m-kubis m-kubis commented May 11, 2018

The animation no longer needs to start from scratch anymore for certain Android versions (#25, #26, maybe #22 and possibly others) in order to work at all.
Fixed issue when only one pulse was moving while the others were not. Also the issue when the animation was not seen.
The behavior should be much more consistent across various versions now.

Tested on 5, 5.1.1, 6, 7, 8, 8.1, P preview

AnimatorSet is no longer used as it seems not to work well with animators having ValueAnimator#setCurrentPlayTime(long) set. Also the actual timing of the call itself relative to starting the animation differs between Android versions.
@kanchanakakans
Copy link

as you have said it is resolved but you have not used this animator set where an option is there to play the animation together ..... have implemented this without animatorset but the effect i found out is single pulse moving forward how can i get the effect of multiple pulse generating with use of Animator in list please Prescribe

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