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

Image flickering when using @cloudinary/react lazyload after adding alt text #234

Open
victorcarvalhosp opened this issue May 6, 2024 · 5 comments

Comments

@victorcarvalhosp
Copy link

victorcarvalhosp commented May 6, 2024

New issue for cloudinary/frontend-frameworks

Before proceeding, please update to the latest version and test if the issue persists

For which package is this issue?

@cloudinary/react

Describe the issue in a sentence or two.

After adding alt text to an image and adding the lazyload plugin, an image flickering is happening very briefly, as it show the alt text before the image. You can see it here if you refresh the demo a few times: https://codesandbox.io/p/sandbox/lazy-loading-lqip-react-js-sdk-2-0-forked-c3hsk6?file=%2Fsrc%2FLazyloading.js%3A34%2C40

I've tried adding a placeholder too and the issue remains the same. using onError to add a default image as shown here doesn't seem to work as well.

Issue Type (Can be multiple)

[ ] Build - Can’t install or import the SDK
[X] Performance - Performance issues
[X] Behaviour - Functions aren’t working as expected
[ ] Documentation - Inconsistency between the docs and behaviour
[ ] Incorrect Types
[ ] Other (Specify)

Steps to reproduce

Just add an image with alt text and lazyload plugin and then refresh the page a few times and you be able to see it a few times before the image loads

<AdvancedImage
              alt="This is the alt text"
              key={i}
              cldImg={cld.image(image)}
              plugins={[lazyload()]}
            ></AdvancedImage>

Error screenshots

image

You can see the "This is the alt text" alt text very quickly before the image loads

Browsers (if issue relates to UI, else ignore)

[] Chrome
[] Firefox
[ ] Safari
[ ] Other (Specify)
[X] All

Versions and Libraries (fill in the version numbers)

package SDK version
Node - 18.15.0
NPM - 9.6.7

Config Files (Please paste the following files if possible)

"dependencies": {
    "@cloudinary/react": "^1.13.0",
    "@cloudinary/url-gen": "^1.19.0",
[...]
}

Repository

https://codesandbox.io/p/sandbox/lazy-loading-lqip-react-js-sdk-2-0-forked-c3hsk6?file=%2Fsrc%2FLazyloading.js%3A33%2C13-38%2C30

@dannyv-cloudinary
Copy link
Contributor

Hi @victorcarvalhosp. Thanks for flagging this with us. I have raised this internally (ref SNI-8022) and we will provide you with updates when we can.

@victorcarvalhosp
Copy link
Author

victorcarvalhosp commented May 19, 2024

Hi @dannyv-cloudinary! Thanks for looking into it. Do you guys have some updates on this? Our site heavily relies on images, and we need lazy load to work properly on it.

Maybe you have some workaround that we cab do in the meantime?

@PixelCook
Copy link

Hi there @victorcarvalhosp. Currently, there is a fix being worked on, and although I don't have a specific ETA I do hope it will be merged soon. I tried tying this ticket to your account but couldn't find your cloud "dessmo". Can you please your cloud name?

@victorcarvalhosp
Copy link
Author

Hi there @victorcarvalhosp. Currently, there is a fix being worked on, and although I don't have a specific ETA I do hope it will be merged soon. I tried tying this ticket to your account but couldn't find your cloud "dessmo". Can you please your cloud name?

Nice, thanks! Our cloud name is "webalys".

@PixelCook
Copy link

Thanks @victorcarvalhosp. Your account is now linked. As soon as we have more info I'll make sure to update you.

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

No branches or pull requests

3 participants