-
Notifications
You must be signed in to change notification settings - Fork 354
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
Example of simple multi-serve setup #69
Comments
hey @madebym I don't have an ipad mini retina :/ But yes, I would also think it took the larger regular one. A fix in the meantime would be adding a retina image(the larger one) for your 900 breakpoint (http://dinbror.dk/blog/blazy/#Retina):
|
Friendly ping @madebym Can you test what screen width it outputs: window.screen.width |
@dinbror Sorry for the late reply. Very confusing things are happening. If I use Correct sizes should be 768 x 1024, nevermind that this is a retina screen and the actual resolution is twice that size. I'll run some more tests, there are many JS ways of checking for width and height. |
Hey @madebym Thanks for the testing results. |
Breakpoints marked as obsolete in version 1.6.0. Use srcset and/or the picture element instead. |
I believe I got the setup correctly, it works as expected. Except one thing, the correct image is not showing on iPad mini retina.
Here is the setup:
Small image is loaded for devices < 900, and regular image that is stored in
data-src
is provided for all other resolutions.This is not working on iPad mini retina(landscape), it shows the smaller image, although clearly it should be serving the regular version through
data-src="regular-image.jpg"
.The text was updated successfully, but these errors were encountered: