Make responsive plugin aware of placeholder to avoid duplicated requests - Githubissues

cloudinary / frontend-frameworks

Cloudinary javascript frontend frameworks SDKs, including Shared HTML layer, Angular, React and Vue SDKs

https://www.cloudinary.com

MIT License

42 stars 16 forks source link

Make responsive plugin aware of placeholder to avoid duplicated requests #223

Closed pawelphilipczyk-cloudinary closed 7 months ago

pawelphilipczyk-cloudinary commented 7 months ago

Pull request for cloudinary/frontend-frameworks

For which package is this PR?

@cloudinary/html

What does this PR solve?

When responsive plugin is combined with placeholder it will fire additional request even before the placeholder image is fetched, see below before and after.
The first request is fired with different analytics than the second one and I'm not sure what is the expected one but we get rid of the first one so the second request (previously third) stays the same

Before

Screenshot 2023-11-28 at 15 48 56

After

Screenshot 2023-11-28 at 15 49 16

Final checklist

[ ] Implementation is aligned to Spec.
[ ] Tests - Add proper tests to the added code.
[ ] Relates to a github issue (link to issue).