Damn!

Hello,

Got an email from google today stating the below message

“Invalid usage of AMP tags (Critical)”.
Details:
Missing URL for attribute ‘src’ in tag ‘amp-img’.

Leaving the src="" has apparently broke an AMP rule. I have now used a transparent image there.

Thanks.

Not a problem. I totally understand. ??

Great. I guess its a win-win. ??

Best,

Ok thank you. For now I have removed the image. Now the default AMP loader is being displayed. I just want to know if I’m breaking any AMP rules by removing that image, meaning leaving the source empty (src="").

One more quick question, I’m having a bit of a trouble replacing the pacman with text? Could use some pointers on that.

@codepeople

Thanks a lot. I just updated the plugin and although google might take a while to recrawl the page, I re-checked on google’s AMP test and the page came out with no errors. I guess now Pages with CFF forms are officially valid AMP pages. ??

Best,

• This reply was modified 6 years, 11 months ago by Damn!.

Can anybody help me here?

Hello,

I see you have released an update to imporve the form’s load time in AMP pages, I’m yet to install the update. However, I have another issue now. I received an email from google stating that

“Google systems have detected that some of your AMP pages do not meet our guidelines; therefore these pages are not eligible for AMP-related features in Google Search.”

“Invalid CSS stylesheet (Critical)”

Details:
The text (CDATA) inside tag ‘style amp-custom’ contains ‘CSS !important’, which is disallowed.
AMP URL:
https://damnripped.com/calculate-body-fat-calculator-included/amp/

Upon viewing the source I see this #cff-form-iframe{margin:0 !important;}

I guess this is what is causing the issue. Please look into this. Thanks.

PS: The error has occured only on this page which has a CFF form within it. All other AMP pages have been indexed without issues.

• This reply was modified 6 years, 11 months ago by Damn!.

Hello again,

I just tried to view the form in a variety of mobile devices and the form is actually loading too too slow, takes almost 20 seconds after the web page loads completely. Is there anything we can do speed up the process?

Also, please let me know how to change the big pacman image to a text stating “Loading, Please wait” or to some other image. Because I don’t want the users to think that there is some kind of issue with the form and leave the site before the form loads.

Thanks.

• This reply was modified 6 years, 11 months ago by Damn!.

Hello @williampatton,

Thanks for the reply. I know that makes total sense. But for some reason forcing https accross the site broke my theme’s design, and the reason I wanted to force https only to AMP pages was because of google’s restriction to only fetch iframes from secure sources and since the AMP pages doesn’t use any of my theme’s templates I thought it would be perfect. But nevermind now as I had fixed the issue that was breaking my theme. I now use https accross the site. ??

Thanks.

Hello,

Thanks for looking into this. Yes, the forms loaded successfully but took a very long time. Is there a way to optimize this?

Also thanks for pointing out the additional shortcode attribute, I’m guessing including it won’t affect the form’s height when the page loads normally?

And I saw something in the AMP docs stating that it’s possible to resize an amp-iframe at runtime? Please see the screen shot: https://prntscr.com/hns62u

Maybe you’ll implement it in a future update, something to look forward to. ??

Also, why is the height set to 500? can it be set to auto? meaning take up the height the form wants?

Thanks.

Hello again,

I just got myself an SSL cert and the pages are now assessible with https, but still the forms won’t load. I get this giant black pacman image, please see the screenshot: https://prntscr.com/hnqkz1. The URL, the plugin tries to load is working fine when I access it in a new tab. (URL: https://www.damnripped.com/?cff-form=45&cff-form-height=500#amp=1)

Ref page: https://damnripped.com/calculate-body-fat-calculator-included/amp/

Thanks for the link to the AMP docs. And I guess you have published the update already, so I’ll just update it. Thank you for the time.

Thanks for looking into this. That piece of code is from an anti-adblock plugin that im using and to be honest I’m frankly surprised to see a function with f-word. I’ll change the script load method and see how it goes. Thanks again.