AMP Issues Detected

Resolved zerojjc
(@zerojjc)

7 years, 2 months ago
Hello,
We are using the Official AMP Plugin for WordPress on this site. I apologize ahead of time for this being my first experience with AMP. The issues we are seeing are as follows:

Errors:
• Disallowed attribute or attribute value present in HTML tag.
• The author stylesheet specified in tag ‘style amp-custom’ is too long – the limit is50000 bytes.
• Custom JavaScript is not allowed.

Warnings:
• Error in required structured data element

I tried searching this forum and Google for answers but I am still unsure how to fix these problems. I used this test: https://search.google.com/test/amp?skip_amp_follow=true&id=gsvAbsw2jQgDcsoHZ0Nkng to see the validation issues but they appear to be within the AMP plugin itself?

I would greatly appreciate any help, and again I apologize for being a newb.
- This topic was modified 7 years, 2 months ago by zerojjc.
The page I need help with: [log in to see the link]

Viewing 4 replies - 1 through 4 (of 4 total)

Plugin Author Weston Ruter
(@westonruter)

7 years, 2 months ago

I’m seeing two validation errors at https://validator.ampproject.org/#url=https%3A%2F%2Fplanetwisebedding.com%2Finspirational-women-fighting-to-save-the-wolrd%2Famp%2F

In particular:

* Custom JavaScript is not allowed.
* The author stylesheet specified in tag ‘style amp-custom’ is too long – document contains 52002 bytes whereas the limit is 50000 bytes.

The second appears due to BigCommerce. There is 46KB of CSS just from that plugin. That’s something you’ll have to report to them.

The other issue may also be due to BigCommerce. What appears aso:

<script async custom-element="amp-mustache" src="https://cdn.ampproject.org/v0/amp-mustache-latest.js"></script>

Should actually be:

<script async custom-template="amp-mustache" src="https://cdn.ampproject.org/v0/amp-mustache-latest.js"></script>

I don’t think these are coming from the AMP plugin, as it generates a script tag like:

<script type='text/javascript' src='https://cdn.ampproject.org/v0/amp-mustache-0.2.js' async custom-template="amp-mustache"></script>

Indeed, I can see that the problem is with \BigCommerce\Amp\Amp_Assets::scripts() in their plugin: https://github.com/bigcommerce/bigcommerce-for-wordpress/blob/cee0338f13818de082349a42336ced8674bc3809/src/BigCommerce/Amp/Amp_Assets.php#L71-L93

I’ll open issues with them to report these issues.

Plugin Author Weston Ruter
(@westonruter)

7 years, 2 months ago

Please follow this pull request: https://github.com/bigcommerce/bigcommerce-for-wordpress/pull/143

Plugin Author Weston Ruter
(@westonruter)

7 years, 2 months ago

Also please follow https://github.com/bigcommerce/bigcommerce-for-wordpress/issues/144

Thread Starter zerojjc
(@zerojjc)

7 years, 2 months ago

Thank you Weston, I will follow those posts you created on github.com and hope they fix the issues.

Viewing 4 replies - 1 through 4 (of 4 total)

The topic ‘AMP Issues Detected’ is closed to new replies.