-
Notifications
You must be signed in to change notification settings - Fork 4
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 size reported as good, but it's not #29
Comments
@nhoizey Presume you're on a retina mac of some sort? The natural width property allows for pixel density |
@andydavies on a Retina MacBook indeed, but the window was on a non Retina external screen. |
|
Actually client hints isn't in play so Cloudinary isn't doing anything clever with DPI, just srcset. |
In any case, we probably need to add a glossary somewhere in the extension so it's clear what the numbers mean. |
@nhoizey can you look if this is still an issue with the 1.5.0 release? thanks |
@tabanliviu sorry for the late answer, the result is almost the same.
So why is |
For example on this page: https://nicolas-hoizey.com/2017/01/how-much-data-should-my-service-worker-put-upfront-in-the-offline-cache.html
When using a viewport width of 614px, it says the screenshot from WebPagetest is at the right dimensions (natural 551x394), but when I use the console to get the
currentSrc
, it shows that the image is 980 pixels wide:The text was updated successfully, but these errors were encountered: