-
-
Notifications
You must be signed in to change notification settings - Fork 108
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
White background #308
Comments
@taasonei I think this result is an expected result because Landscapist doesn't compose everything from the first, it observes the state and compose each different composables depending on its state. Let me think if there's a great way to make the transition smooth with fade animation or something. |
@taasonei I'm wondering what if you use the Crossfade(targetState = imageRes, label = "") { image ->
CoilImage(
)
.. |
@skydoves looks like there is no difference when I add |
any update on this issue? |
A similar thing is happening for me. It's a gray box that appears every time before the loading preview appears. There's no error placeholder being shown. It's especially jarring while scrolling in a grid with multiple items because you see a lot of gray boxes really quickly and then the loading state/image. I think this could be the same issue but our apps have different default color schemes? Not sure though |
Hey @JayyyR, sorry for the delayed response. Do you use a shimmering effect or what placeholder do you use? It would be really helpful to debug if you could share the |
I use a shimmer effect with landscapist-coil and I see this problem. The first time that this screen is opened, there's the regular shimmer (for a few seconds) because the image is being loaded from the network. Next time I open this screen, I see is a brief shimmer appears in the UI before disappearing. It looks like the shimmer is being applied for a very brief duration (perhaps less than 100ms), but in reality the shimmer was not actually needed (because the image was fetched from cache). Here's the code I use (landscapist 2.2.13):
|
@curioustechizen that sounds very fair. Thank you for the details! Let me fix this in the next stable release. |
Please complete the following information:
Library Version [v2.2.2]
implementation 'com.github.skydoves:landscapist-glide:2.2.2'
implementation 'com.github.skydoves:landscapist-coil:2.2.2'
implementation 'com.github.skydoves:landscapist-placeholder:2.2.2'
Affected Device(s) [Not a device specific]
Describe the Bug:
I've tried
GlideImage
andCoilImage
. Both show white background when image loading. It appears before loading placeholder:error placeholder -> white background -> loading placeholder -> loaded image
There is no white background when using coil or glide compose instead
Sample code
Video sample (slowing)
lv_0_20230711120747.mp4
Expected Behavior:
error placeholder -> loading placeholder -> loaded image
The text was updated successfully, but these errors were encountered: