egornmore

@milindmore22

Still validation status failed, but plugin shows 0 errors.

@milindmore22
Tried to turn off plugins that are missing on Staging that didn’t work. Could it be a server-side or non AMP CDN issue by any chance?

• This reply was modified 2 years, 7 months ago by egornmore.
• This reply was modified 2 years, 7 months ago by egornmore.

Snippet: https://goonlinetools.com/snapshot/code/#9xv2kewg9mbknjcvtdv8v
URL: https://staging.nmore.dev/

@milindmore22
Removed layout, still has validation errors. Reinstallation of AMP Plugin with “Delete plugin data at uninstall” ticked, unfortunately did not help as well

• This reply was modified 2 years, 7 months ago by egornmore.
• This reply was modified 2 years, 7 months ago by egornmore.

@milindmore22
Tested exactly the same menu on staging and live websites, on staging it has no validation errors, and on the live website, it has. Disabling AMP Optimizer server-side rendering did not help. I haven’t added <i-amphtml-sizer> manually, all images in the <amp-mega-menu> are added as <amp-img> e.g. <amp-img src="..." width="32" height="32" alt="Slack" layout="intrinsic"></amp-img>

@milindmore22
I’ve done tests on staging domain which is supposed to be almost identical to the live website, as I thought… Now I realised that the live website has validation errors, sorry… I will double-check everything and will get back to you with an update.

• This reply was modified 2 years, 7 months ago by egornmore.

Hello @milindmore22

Added filter, checked on AMP validator, it says – “Validation Status: PASS”

Screenshot: https://ibb.co/Wc2brVV

Alright, thanks for the fast response!

Updating CF7 solved the issue

It works
Thanks for your time!

Hi @jadealombro

I’ll try to play around and use <amp-sidebar> instead.

Thanks for your time!

Hello @jadealombro,
I am using the official Google AMP plugin By AMP Project Contributors.

I apologize for opening multiple threads.
I removed the following part in “class-frontend.php”:

if ( wpforms_is_amp() ) {

  $state = array(
    'submitting' => false,
  );

  printf(
    '<amp-state id="%s"><script type="application/json">%s</script></amp-state>',
    $this->get_form_amp_state_id( $form_id ),
    wp_json_encode( $state )
  );

}

Will my wpform work without those lines? And if not, can I add amp-state after closing <amp-mega-menu> manually as static?

• This reply was modified 3 years, 1 month ago by egornmore.
• This reply was modified 3 years, 1 month ago by egornmore.
• This reply was modified 3 years, 1 month ago by egornmore.
• This reply was modified 3 years, 1 month ago by egornmore.

The validation error occurs if the wp forms shortcode is in the <amp-mega-menu> component.

I found a fast-fix solution.
Filled all necessary theme meta fields in “Custom Fields”.
https://ibb.co/5FyHBmd

• This reply was modified 3 years, 3 months ago by egornmore.