FormidableLabs / react-progressive-image

React component for progressive image loading
735 stars 61 forks source link

Image is loaded twice #32

Open ioanlucut opened 5 years ago

ioanlucut commented 5 years ago

First of all, thank you for the great library.

I integrated it in one of our apps and I noticed that by using the basic example, the image gets loaded twice.

Is this some kind of expected behavior?

Thank you.

hilja commented 5 years ago

Was wondering the same just.

Loque- commented 5 years ago

I thought it was, but it's only if you have the cache disabled in devtools that this behaviour exists.

morganfeeney commented 4 years ago

I'm seeing this too, if I run the code in a React app the image is loaded twice, however if I grab the output html and run it in a static html file the problem does not occur.

sanishkr commented 4 years ago

One of the call is made by dist.js which is this library and another one is made by react-dom.development.js which I think is called only in dev mode or when network cache is disabled in chrome dev tools as mentioned by @Loque-

I thought it was, but it's only if you have the cache disabled in devtools that this behaviour exists.

mattg66 commented 4 years ago

Hi, I am also having this issue. Cache is not disabled in dev tools, and image is being loaded by both dist.js and react-dom.production.min.js

endrureza commented 4 years ago

has this issue been solved?

Loque- commented 4 years ago

From what I can tell it's not an issue. Without a test case to demonstrate it, the GitHub issue should probably be closed to prevent people thinking it is still a problem.

adshrc commented 4 years ago

For me it was just the case while dev tools cache was disabled. Thanks!