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
fixes FadeInImage for an edge case #111035
fixes FadeInImage for an edge case #111035
Conversation
7da5c71
to
5b95b91
Compare
5b95b91
to
f0af5b0
Compare
…y loaded but second isn't
f0af5b0
to
0c2af87
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, just some nits
cc @dnfield for secondary review |
Co-authored-by: chunhtai <47866232+chunhtai@users.noreply.github.com>
Sorry, but is it known when this will get released on stable? On Flutter 3.3.3 this is still a problem and I was wondering how to work around that. |
Can we have this in stable please? We had to downgrade to 3.0.0 to avoid this issue. |
requesting cp. See the original issue |
where first image was synchronously loaded but second isn't.
when first image is precached and then the image is changed to an uncached one, placeholder's opacity animation was never initialized but the code assumed it'll be if the image is not synchronously loaded.
adds regression test for it.
fixes #111011
caused by #94601
Pre-launch Checklist
///
).If you need help, consider asking for advice on the #hackers-new channel on Discord.