Please only file reports about bugs in AMP here.
If you have a bug for AMP please fill in the following template. Delete everything except the headers (including this text).
We are in the process of developing our new item page using the AMP technology and here is the development version.
http://www.sampathmadhuranga.com/projects/superjeweler/amp/item-detail.html
If you load the above URL you will notice that, the initail image doesnt load on the first paint.
But when you click on a thumbnail image it does.
Please provide a public URL and ideally a reduced test case (e.g. on jsbin.com) that exhibits only your issue and nothing else. Provide step-by-step instructions for reproducing the issue:
All Browsers
All browsers? Some specific browser? What device type?
latest
Is this a new issue? Or was it always broken? Paste the version of AMP where you saw this issue. (You can find the version printed in your browser's console.)
@Suranga-Aluthgamage Thanks for the issue! π Could you please fill out the form above? Not quite sure what the issue is.
Thanks! π
Issue is : "If you load the above URL you will notice that, the initail image doesnt load on the first paint.
But when you click on a thumbnail image it does." Here is the dev URL
http://www.sampathmadhuranga.com/projects/superjeweler/amp/item-detail.html
layout=responsive isn't supported on <amp-carousel>. This is likely the reason.
@Suranga-Aluthgamage please see @jridgewell and let us know if this fixes the issue for you π
Thanks! We will let you know if this solves the issue.
Really appreciate the support.
Suranga
From: Justin Ridgewell notifications@github.com
Sent: Tuesday, April 9, 2019 12:58 AM
To: ampproject/amphtml amphtml@noreply.github.com
Cc: Suranga-Aluthgamage suranga@hansadiamonds.com; Mention mention@noreply.github.com
Subject: Re: [ampproject/amphtml] Initial image Loading Issue (#21739)
layout=responsive isn't supported on
β
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/ampproject/amphtml/issues/21739#issuecomment-480973395 , or mute the thread https://github.com/notifications/unsubscribe-auth/AvCAGknDxw11YSWwcVv6KNF_8N21RAEuks5ve5hLgaJpZM4cfLsb . https://github.com/notifications/beacon/AvCAGh8y_rXdxvFH0obcua-hPPuQr9xLks5ve5hLgaJpZM4cfLsb.gif
This email has been checked for viruses by Avast antivirus software.
https://www.avast.com/antivirus
Sure. I'll check.
Thanks.
On Tue, Apr 9, 2019, 08:59 suranga@hansadiamonds.com wrote:
Thanks! We will let you know if this solves the issue.
Really appreciate the support.
Suranga
From: Justin Ridgewell notifications@github.com
Sent: Tuesday, April 9, 2019 12:58 AM
To: ampproject/amphtml amphtml@noreply.github.com
Cc: Suranga-Aluthgamage suranga@hansadiamonds.com; Mention <
[email protected]>
Subject: Re: [ampproject/amphtml] Initial image Loading Issue (#21739)layout=responsive isn't supported on
. This is likely the
reason.β
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ampproject/amphtml/issues/21739#issuecomment-480973395,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AvCAGknDxw11YSWwcVv6KNF_8N21RAEuks5ve5hLgaJpZM4cfLsb
.https://www.avast.com/sig-email?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=emailclient&utm_term=icon Virus-free.
www.avast.com
https://www.avast.com/sig-email?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=emailclient&utm_term=link
<#m_9037994952406829493_DAB4FAD8-2DD7-40BB-A1B8-4E2AA1F9FDF2>
Hi,
I have tried the solution in github, but it is not worked for our issue.
http://www.sampathmadhuranga.com/projects/superjeweler/amp/item-detail.html
Thanks.
On Tue, Apr 9, 2019 at 12:50 PM Vithanage Sampath <
[email protected]> wrote:
Sure. I'll check.
Thanks.
On Tue, Apr 9, 2019, 08:59 suranga@hansadiamonds.com wrote:
Thanks! We will let you know if this solves the issue.
Really appreciate the support.
Suranga
From: Justin Ridgewell notifications@github.com
Sent: Tuesday, April 9, 2019 12:58 AM
To: ampproject/amphtml amphtml@noreply.github.com
Cc: Suranga-Aluthgamage suranga@hansadiamonds.com; Mention <
[email protected]>
Subject: Re: [ampproject/amphtml] Initial image Loading Issue (#21739)layout=responsive isn't supported on
. This is likely the
reason.β
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ampproject/amphtml/issues/21739#issuecomment-480973395,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AvCAGknDxw11YSWwcVv6KNF_8N21RAEuks5ve5hLgaJpZM4cfLsb
.https://www.avast.com/sig-email?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=emailclient&utm_term=icon Virus-free.
www.avast.com
https://www.avast.com/sig-email?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=emailclient&utm_term=link
<#m_-1479185145787251564_m_9037994952406829493_DAB4FAD8-2DD7-40BB-A1B8-4E2AA1F9FDF2>
@Suranga-Aluthgamage
So I looked at your website:

And it seems that the amp-carousel is still layout=responsive. Let me know if I am testing this correctly.
Thank you! π
Hi Aaron,
Ok I will get this double checked and get back to you.
Appreciate your support.
Thanks and Regards
Suranga
From: Aaron Turner notifications@github.com
Sent: Thursday, April 11, 2019 5:45 AM
To: ampproject/amphtml amphtml@noreply.github.com
Cc: Suranga-Aluthgamage suranga@hansadiamonds.com; Mention mention@noreply.github.com
Subject: Re: [ampproject/amphtml] Initial image Loading Issue (#21739)
@Suranga-Aluthgamage https://github.com/Suranga-Aluthgamage
So I looked at your website:
https://user-images.githubusercontent.com/1448289/55921981-0ead7380-5bb4-11e9-9ed4-85f241497bed.png
And it seems that the amp-carousel is still layout=responsive. Let me know if I am testing this correctly.
Thank you! π
β
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/ampproject/amphtml/issues/21739#issuecomment-481917905 , or mute the thread https://github.com/notifications/unsubscribe-auth/AvCAGo7UUiCUYRSRRgulAOMb28xo4jJBks5vfn6CgaJpZM4cfLsb . https://github.com/notifications/beacon/AvCAGuwuyVpWD_fjrFssuZMcTQF2Aa8Rks5vfn6CgaJpZM4cfLsb.gif
This email has been checked for viruses by Avast antivirus software.
https://www.avast.com/antivirus
Hi,
According to my knowledge layout='responsive' is supported for
amp-carousel,
which has type=slides.
And also I have compared my code with a example in amp official site. I
couldn't find any difference there.
Below I have attached screens of my code and theirs. Please find the
attachments.
Thanks.
Amp code -
[image: amp-code.JPG]
*My code - *
[image: my-code..JPG]
On Thu, Apr 11, 2019 at 9:09 AM suranga@hansadiamonds.com wrote:
Hi Aaron,
Ok I will get this double checked and get back to you.
Appreciate your support.
Thanks and Regards
Suranga
From: Aaron Turner notifications@github.com
Sent: Thursday, April 11, 2019 5:45 AM
To: ampproject/amphtml amphtml@noreply.github.com
Cc: Suranga-Aluthgamage suranga@hansadiamonds.com; Mention <
[email protected]>
Subject: Re: [ampproject/amphtml] Initial image Loading Issue (#21739)@Suranga-Aluthgamage https://github.com/Suranga-Aluthgamage
So I looked at your website:
[image: Screen Shot 2019-04-10 at 5 13 36 PM]
https://user-images.githubusercontent.com/1448289/55921981-0ead7380-5bb4-11e9-9ed4-85f241497bed.pngAnd it seems that the amp-carousel is still layout=responsive. Let me
know if I am testing this correctly.Thank you! π
β
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ampproject/amphtml/issues/21739#issuecomment-481917905,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AvCAGo7UUiCUYRSRRgulAOMb28xo4jJBks5vfn6CgaJpZM4cfLsb
.https://www.avast.com/sig-email?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=emailclient&utm_term=icon Virus-free.
www.avast.com
https://www.avast.com/sig-email?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=emailclient&utm_term=link
<#m_-3950571827005141894_DAB4FAD8-2DD7-40BB-A1B8-4E2AA1F9FDF2>
Hi,
I changed my code as below but every time this issue occurs.
Change 1 - Here I have replaced 'responsive' by 'fixed-height'.The Issue
is same as earlier.
layout="fixed-height"
type="slides"
on="slideChange:carouselWithPreviewSelector.toggle(index=event.index, Change 2 - Here I have removed *layout *property. Carousel didn't work on="slideChange:carouselWithPreviewSelector.toggle(index=event.index, If you have any suggestion or if you see any wrong in my changes please let I am very appreciating you help on this issue π Thanks. On Thu, Apr 11, 2019 at 10:34 AM Vithanage Sampath < Hi, According to my knowledge layout='responsive' is supported for Below I have attached screens of my code and theirs. Please find the Thanks. Amp code - *My code - * On Thu, Apr 11, 2019 at 9:09 AM suranga@hansadiamonds.com wrote: Hi Aaron, Ok I will get this double checked and get back to you. Appreciate your support. Thanks and Regards Suranga From: Aaron Turner notifications@github.com @Suranga-Aluthgamage https://github.com/Suranga-Aluthgamage So I looked at your website: [image: Screen Shot 2019-04-10 at 5 13 36 PM] And it seems that the amp-carousel is still layout=responsive. Let me Thank you! π β https://www.avast.com/sig-email?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=emailclient&utm_term=icon Virus-free.
value=true)">
properly.
height="300"
type="slides"
value=true)">
me know.
[email protected]> wrote:
amp-carousel,
which has type=slides.
And also I have compared my code with a example in amp official site. I
couldn't find any difference there.
attachments.
[image: amp-code.JPG]
[image: my-code..JPG]
Sent: Thursday, April 11, 2019 5:45 AM
To: ampproject/amphtml amphtml@noreply.github.com
Cc: Suranga-Aluthgamage suranga@hansadiamonds.com; Mention <
[email protected]>
Subject: Re: [ampproject/amphtml] Initial image Loading Issue (#21739)
https://user-images.githubusercontent.com/1448289/55921981-0ead7380-5bb4-11e9-9ed4-85f241497bed.png
know if I am testing this correctly.
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ampproject/amphtml/issues/21739#issuecomment-481917905,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AvCAGo7UUiCUYRSRRgulAOMb28xo4jJBks5vfn6CgaJpZM4cfLsb
.
www.avast.com
https://www.avast.com/sig-email?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=emailclient&utm_term=link
<#m_-6792497411155754405_m_-3950571827005141894_DAB4FAD8-2DD7-40BB-A1B8-4E2AA1F9FDF2>
@Suranga-Aluthgamage Thanks for making those changes! Surprised it didn't work.
@jridgewell
I went ahead and made my own example on glitch, and it seems the issue is that the amp-img never loads:

Since I know @sparhami is working on carousel, I'll re-assign to them since it may be a UI issue.
cc @aghassemi
Also, @Suranga-Aluthgamage is correct in that, if you have type="slides" amp-carousel should allow layout="responsive". See: https://amp.dev/documentation/components/amp-carousel?referrer=ampproject.org
Hi Aaron,
Good luck and let us know once you figure out the solution.
Appreciate your support.
Thanks and Regards
Suranga
From: Aaron Turner notifications@github.com
Sent: Friday, April 12, 2019 1:52 AM
To: ampproject/amphtml amphtml@noreply.github.com
Cc: Suranga-Aluthgamage suranga@hansadiamonds.com; Mention mention@noreply.github.com
Subject: Re: [ampproject/amphtml] Initial image Loading Issue (#21739)
@Suranga-Aluthgamage https://github.com/Suranga-Aluthgamage Thanks for making those changes! Surprised it didn't work.
@jridgewell https://github.com/jridgewell
I went ahead and made my own example on glitch, and it seems the issue is that the amp-img never loads:
https://user-images.githubusercontent.com/1448289/55990482-a070bb80-5c5c-11e9-9e5e-7c7f05040764.png
Since I know @sparhami https://github.com/sparhami is working on carousel, I'll re-assign to them since it may be a UI issue.
cc @aghassemi https://github.com/aghassemi
β
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/ampproject/amphtml/issues/21739#issuecomment-482296588 , or mute the thread https://github.com/notifications/unsubscribe-auth/AvCAGuKaTTnHMO_26PZnKM2qcbFeeHMcks5vf5lvgaJpZM4cfLsb . https://github.com/notifications/beacon/AvCAGhneTVkMpDPs-AOodYuLXEyEUc7Qks5vf5lvgaJpZM4cfLsb.gif
This email has been checked for viruses by Avast antivirus software.
https://www.avast.com/antivirus
Dear Aaron
Any update on the initial image loading issue?
For your information, we have completed our AMPed Deal of the day page and due to this loading issue we are unable to launch it.
And same for our main item page. This one of the major issue for us. And also please let me know if you can suggest an alternative.
Really appreciate your support.
And also I went through a video from the AMP 2019 conference, there I got to know that AMP will be compatible with most of the jQuery Libraries.
Will it be related to a next AMP version?
Thanks and Regards
Suranga
From: Aaron Turner notifications@github.com
Sent: Friday, April 12, 2019 1:52 AM
To: ampproject/amphtml amphtml@noreply.github.com
Cc: Suranga-Aluthgamage suranga@hansadiamonds.com; Mention mention@noreply.github.com
Subject: Re: [ampproject/amphtml] Initial image Loading Issue (#21739)
@Suranga-Aluthgamage https://github.com/Suranga-Aluthgamage Thanks for making those changes! Surprised it didn't work.
@jridgewell https://github.com/jridgewell
I went ahead and made my own example on glitch, and it seems the issue is that the amp-img never loads:
https://user-images.githubusercontent.com/1448289/55990482-a070bb80-5c5c-11e9-9e5e-7c7f05040764.png
Since I know @sparhami https://github.com/sparhami is working on carousel, I'll re-assign to them since it may be a UI issue.
cc @aghassemi https://github.com/aghassemi
β
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/ampproject/amphtml/issues/21739#issuecomment-482296588 , or mute the thread https://github.com/notifications/unsubscribe-auth/AvCAGuKaTTnHMO_26PZnKM2qcbFeeHMcks5vf5lvgaJpZM4cfLsb . https://github.com/notifications/beacon/AvCAGhneTVkMpDPs-AOodYuLXEyEUc7Qks5vf5lvgaJpZM4cfLsb.gif
This email has been checked for viruses by Avast antivirus software.
https://www.avast.com/antivirus
/to @cathyxz, can you look into why <amp-pan-zoom> isn't calling layoutCallback on its child <amp-img>?
Hi Sampath
Please see below and do the needful and update the team.
Thanks and Regards
Suranga
From: Justin Ridgewell notifications@github.com
Sent: Wednesday, April 24, 2019 10:01 PM
To: ampproject/amphtml amphtml@noreply.github.com
Cc: Suranga-Aluthgamage suranga@hansadiamonds.com; Mention mention@noreply.github.com
Subject: Re: [ampproject/amphtml] Initial image Loading Issue (#21739)
/to @cathyxz https://github.com/cathyxz , can you look into why
β
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/ampproject/amphtml/issues/21739#issuecomment-486317990 , or mute the thread https://github.com/notifications/unsubscribe-auth/ALYIAGWBZRHALXNQ2WCZNFDPSCDLXANCNFSM4HD4XMNQ . https://github.com/notifications/beacon/ALYIAGSXBHL5ERLEME26J53PSCDLXANCNFSM4HD4XMNQ.gif
This email has been checked for viruses by Avast antivirus software.
https://www.avast.com/antivirus
Looks like a problem with the way ownership stuff works. Carousel takes ownership of direct children (amp-pan-zoom) here and lays it out but runtime does not go further to layout children of amp-pan-zoom. amp-pan-zoom also does not do this an expect children to be laid out via runtime.
I think it makes sense for amp-pan-zoom to always directly schedule layout for its children regardless and not rely on resource manager to do it since it is a container element (by just calling scheduleLayout(this.contiainer_) inside its own layout callback. Similar to how other container components such as flying-carpet an sidebar do this.
@jridgewell Does this sound reasonable?
(never mind, thats what Justin said here)
@cathyxz added it to FixIt bucket.
FixedIt. xD
Hi Guys, when can I expect the solution for my original issue?
Hi
Sorry I donβt understand. Is my original issue solved?
Please let me know.
Thanks and Regards
Suranga
From: Cathy Zhu notifications@github.com
Sent: Friday, April 26, 2019 11:35 PM
To: ampproject/amphtml amphtml@noreply.github.com
Cc: Suranga-Aluthgamage suranga@hansadiamonds.com; Mention mention@noreply.github.com
Subject: Re: [ampproject/amphtml] amp-pan-zoom in amp-carousel Image loading issue (#21739)
Closed #21739 https://github.com/ampproject/amphtml/issues/21739 via #21996 https://github.com/ampproject/amphtml/pull/21996 .
β
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/ampproject/amphtml/issues/21739#event-2303815794 , or mute the thread https://github.com/notifications/unsubscribe-auth/ALYIAGVPTFS2H4A57XLJFATPSM74RANCNFSM4HD4XMNQ . https://github.com/notifications/beacon/ALYIAGQFJTGC4QB7V3V2I4DPSM74RANCNFSM4HD4XMNQ.gif
This email has been checked for viruses by Avast antivirus software.
https://www.avast.com/antivirus
Yes, it is fixed and will be in production release in less than two weeks.
Hi Ali
How do we get informed once its released? Will you guys send an updated email?
And also should I make any changes to the current page after you fixed the issue? Any version updates should be done from my end?
Please advise.
Thanks and Regards
Suranga
From: Ali Ghassemi notifications@github.com
Sent: Monday, April 29, 2019 8:01 PM
To: ampproject/amphtml amphtml@noreply.github.com
Cc: Suranga-Aluthgamage suranga@hansadiamonds.com; Mention mention@noreply.github.com
Subject: Re: [ampproject/amphtml] amp-pan-zoom in amp-carousel Image loading issue (#21739)
Yes, it is fixed and will be in production release in less than two weeks.
β
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/ampproject/amphtml/issues/21739#issuecomment-487602759 , or mute the thread https://github.com/notifications/unsubscribe-auth/ALYIAGSW32MEPGWJV2DMQ3LPS4BBVANCNFSM4HD4XMNQ . https://github.com/notifications/beacon/ALYIAGQ532ODIFUXSANC3T3PS4BBVANCNFSM4HD4XMNQ.gif
This email has been checked for viruses by Avast antivirus software.
https://www.avast.com/antivirus
We announce releases on the #release channel on Slack. Schedule is and more details also documented here https://github.com/ampproject/amphtml/blob/e52c0ce99f4b06fae7c8af3433735fc0024c54c1/contributing/release-schedule.md#detailed-schedule
You do not have to do anything on your page.
Hi Guys
From 2019 conf - Got to know that amp-script will support third party JS? Can we try that now or not yet launched?
Any examples with amp third party JS?
Thanks and Regards
Suranga
From: Ali Ghassemi notifications@github.com
Sent: Monday, April 29, 2019 10:00 PM
To: ampproject/amphtml amphtml@noreply.github.com
Cc: Suranga-Aluthgamage suranga@hansadiamonds.com; Mention mention@noreply.github.com
Subject: Re: [ampproject/amphtml] amp-pan-zoom in amp-carousel Image loading issue (#21739)
We announce releases on the #release channel on Slack. Schedule is and more details also documented here https://github.com/ampproject/amphtml/blob/e52c0ce99f4b06fae7c8af3433735fc0024c54c1/contributing/release-schedule.md#detailed-schedule
You do not have to do anything on your page.
β
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/ampproject/amphtml/issues/21739#issuecomment-487649194 , or mute the thread https://github.com/notifications/unsubscribe-auth/ALYIAGU4WXDTPGWNBE4OB3DPS4PA7ANCNFSM4HD4XMNQ . https://github.com/notifications/beacon/ALYIAGTP27XBTLYDS7VAJJ3PS4PA7ANCNFSM4HD4XMNQ.gif
This email has been checked for viruses by Avast antivirus software.
https://www.avast.com/antivirus
@Suranga-Aluthgamage checkout https://github.com/ampproject/amphtml/blob/master/extensions/amp-script/amp-script.md . It is still an experimental feature.
Our Slack channel would be a better medium for questions. Please join https://bit.ly/amp-slack-signup
Hi Guys
This is to inform you guys that the initial issue has been fixed and now the initial image is loading fine.
I think with the latest release. Really appreciate your service!
Thanks and Regards
Suranga
From: [email protected] suranga@hansadiamonds.com
Sent: Tuesday, April 30, 2019 10:35 PM
To: 'ampproject/amphtml' reply@reply.github.com; 'ampproject/amphtml' amphtml@noreply.github.com
Cc: 'Mention' mention@noreply.github.com
Subject: RE: [ampproject/amphtml] amp-pan-zoom in amp-carousel Image loading issue (#21739)
Hi Guys
From 2019 conf - Got to know that amp-script will support third party JS? Can we try that now or not yet launched?
Any examples with amp third party JS?
Thanks and Regards
Suranga
From: Ali Ghassemi <[email protected] notifications@github.com >
Sent: Monday, April 29, 2019 10:00 PM
To: ampproject/amphtml <[email protected] amphtml@noreply.github.com >
Cc: Suranga-Aluthgamage <[email protected] suranga@hansadiamonds.com >; Mention <[email protected] mention@noreply.github.com >
Subject: Re: [ampproject/amphtml] amp-pan-zoom in amp-carousel Image loading issue (#21739)
We announce releases on the #release channel on Slack. Schedule is and more details also documented here https://github.com/ampproject/amphtml/blob/e52c0ce99f4b06fae7c8af3433735fc0024c54c1/contributing/release-schedule.md#detailed-schedule
You do not have to do anything on your page.
β
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/ampproject/amphtml/issues/21739#issuecomment-487649194 , or mute the thread https://github.com/notifications/unsubscribe-auth/ALYIAGU4WXDTPGWNBE4OB3DPS4PA7ANCNFSM4HD4XMNQ . https://github.com/notifications/beacon/ALYIAGTP27XBTLYDS7VAJJ3PS4PA7ANCNFSM4HD4XMNQ.gif
This email has been checked for viruses by Avast antivirus software.
https://www.avast.com/antivirus