Skip to content
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

AMP images not loading after few(1000px) from the top of browser #25555

Closed
satishsurve opened this issue Nov 12, 2019 · 6 comments
Closed

AMP images not loading after few(1000px) from the top of browser #25555

satishsurve opened this issue Nov 12, 2019 · 6 comments

Comments

@satishsurve
Copy link

satishsurve commented Nov 12, 2019

What's the issue?

AMP Images not loading after the first few hundred pixels from the top of the browser. Initial viewport(top 1000px) sections images are loading correctly but when we scroll down page (after some 1000px) not loading AMP images.

As per my observation after AMP page rendering amp-image tag should be converted to like this -
image

but getting AMP image tag like this after top (1000 px) -
image
image

When we resize the page, AMP images loading correctly.

What browsers are affected?

Chrome - Version 78.0.3904.97 (Official Build) (64-bit)

Which AMP version is affected?

using https://cdn.ampproject.org/v0.js

@alanorozco
Copy link
Member

@satishsurve Hi, which runtime version are you running? Your console should print it as part of the first message.

@satishsurve
Copy link
Author

@alanorozco Thanks for your quick response.
We are using runtime version is - 1911062056110

@jridgewell
Copy link
Contributor

Do you have a demo page?

@satishsurve
Copy link
Author

satishsurve commented Nov 21, 2019

@jridgewell , @cathyxz
Please find demo on below link

https://satishsurve.github.io/

Just to remind: open above URL in mobile and try to scroll to reproduce the issue.

Also you can check source code on below link
https://github.com/satishsurve/satishsurve.github.io/blob/master/index.html

Thanks!

@jridgewell
Copy link
Contributor

Ahh, I see. You're using an overflow: scroll container, which AMP doesn't yet directly support. We have an experimental build that enables the feature, you can see instructions at #24929 (comment).

Closing this a duplicate of #24929.

@satishsurve
Copy link
Author

Thanks! @jridgewell :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants