Not a valid AMP page

Hi @theman77,
Thanks for your detailed report. Could you please share the URL where this occurs, assuming it’s on a public site?

The col element is only supposed to allow a span attribute: https://github.com/ampproject/amphtml/blob/8ead38074d31017c79bfde97d898f8b530f42ef7/validator/validator-main.protoascii#L3124-L3128

The whitelist sanitizer may be incorrectly merging the width layout attribute among the attributes allowed on an element: https://github.com/Automattic/amp-wp/blob/4e1cd227b10f2fcd6add44a24b078158aad41e53/includes/sanitizers/class-amp-allowed-tags-generated.php#L10573

See here: https://github.com/Automattic/amp-wp/blob/4e1cd227b10f2fcd6add44a24b078158aad41e53/includes/sanitizers/class-amp-tag-and-attribute-sanitizer.php#L1468

if ( isset( $this->globally_allowed_attributes[ $attr_name ] ) || isset( $this->layout_allowed_attributes[ $attr_name ] ) || isset( $attr_spec_list[ $attr_name ] ) ) {

layout_allowed_attributes is not globally-allowed it seems. Maybe that condition needs to be removed.

thanks, where do i need to remove this?

@theman77 It’s not something you can remove. It’s something we need to fix in the plugin, potentially. But in order to confirm you need to share the AMP URL for the page that has the problem.

rebrand.ly/jlksdjf23

Thanks. So the non-AMP version of the page contains a table:

<table class="auto-style1 sws_custom_table" style="width: 100%; display: table; background-color: #ffffff; border: 2px solid #eaeaea;" border="0" width="630" cellspacing="0"><colgroup> <col style="mso-width-source: userset; mso-width-alt: 8988; width: 190pt;" width="253" /> <col style="mso-width-source: userset; mso-width-alt: 6826; width: 144pt;" width="192" /> <col style="mso-width-source: userset; mso-width-alt: 8135; width: 172pt;" width="229" /></colgroup>
...</table>

This was pasted from Microsoft Word apparently.

The immediate fix is for you to remove the col tags, but the fix for the plugin is to convert width attribute to a width style, perhaps, so that it will be combined with the other styles which are moved to a style rule.

I’ve opened an issue in the plugin repo to implement this latter fix: https://github.com/Automattic/amp-wp/issues/1062

Also, please note that the col element’s width attribute is now obsolete as of HTML5: https://developer.mozilla.org/en-US/docs/Web/HTML/Element/col