This comment has been minimized.

@rase- The images still show up, but Retina versions aren't being served to HiDPI screens when Lazy Load is active. So the images look a bit blurry. If I disable Lazy Load, then Photon will load in Retina versions as expected.

This comment has been minimized.

@ChrisBegley your suspicion seems correct. I beutified devicepx-jetpack.js and took a peek. It does do scale detection, and does only look at the src attribute.

A recent commit in Jetpack delays the call that will enable scaling when using lazy loading (ffa1b1d). Have you by any chance tried the current master branch or Jetpack, and if you haven't do you have a development setup where you could?

I suspect that the mentioned change might also fix this issue, and it would be extremely helpful if you could report your findings regarding your specific situation.

This comment has been minimized.

@ChrisBegley thanks for trying it out. I'll look further into it. I think I can imitate your experience with retina versions at least to some extent via zooming in my browser. I'll get back to you when I figure something out.

This comment has been minimized.

This comment has been minimized.

According my current understanding, the reason why the zoom parameter isn't properly set for you is that devicepx-jetpack.js will set scale="0" to all images, which indicates disabling scaling because it looks at the placeholder image instead of the actual source, which doesn't match any of the predefined formats to be handled.

The reason the placeholder image doesn't match any of the pre-existing rules is that the placeholder image doesn't get passed through Photon for you. When trying things out in my test environment, they do. Also for me everything will have the data-recalc-dims attribute. If these would appear for you too, then the simple solution suggested in #530 would be enough.

I will investigate Photon further tomorrow and try to find out more about the best way to deal with both of the cases.

we avoid both this problem, and the problem presented in #530. The above code example has added an additional check: if no scale has been set yet, and we have a data-lazy-src attribute indicating some form of Photon supported lazy loading, let's postpone the scaling until the src attribute has the same content as the data-lazy-src attribute.

This comment has been minimized.

I imitated the reported situation by zooming in in my browser, and hosting a version of the script where the zoomImages function is replaced with the above code, and changed the enqueue to the following