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

fixed bug in android 4.1.1 (team-supercharge/ShimmerLayout#66). #67

Merged
merged 1 commit into from Nov 8, 2018

Conversation

gaulazat
Copy link

in ShimmerLayout is used ComposeShader, which contains bug in android 4.1.1 with layer hardware acceleration: https://stackoverflow.com/questions/12445583/issue-with-composeshader-on-android-4-1-1

@veghtomi
Copy link
Collaborator

I read the link you have attached. It is really interesting that it happens only at API level 16, and not below or above. I still investigate this, but if I do not find another solution, your pull request will be accepted.

@chichkanov
Copy link

So is there any other way to fix this issue?

@veghtomi
Copy link
Collaborator

veghtomi commented Nov 6, 2018

@chichkanov Just checked yesterday. Your solution will be great. One thing, could you move the enableForcedSoftwareLayerIfNeeded() method before the start animation call? Now it is called after that

@gaulazat
Copy link
Author

gaulazat commented Nov 7, 2018

@veghtomi Probably you wanted to write to me. Well, I fixed the PR after your comment.

@veghtomi veghtomi merged commit bd48255 into team-supercharge:master Nov 8, 2018
@veghtomi
Copy link
Collaborator

veghtomi commented Nov 8, 2018

@gali-azat96 Thank you very much for your remark and the quick fix as well!! I plan to have it released by the end of November.

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.

None yet

3 participants