Forum Replies Created
-
AuthorPosts
-
Hi Ismael,
Yes they are mostly product pages.
As I said above, changing the gallery did not have much of an impact when I did a test. I don’t believe the Woocommerce gallery images are not lazy loaded, not sure how you’re seeing this?
I am happy to try it globally for a few weeks, but I’d need a fix that stops all images on the page (inc images/videos on the footer) appearing in the default gallery widget as I mentioned above.
Cheers,
JasonThanks Ismael. Do you have any tips for FCP? I am trying to make sure my site is ready for mobile first indexing. My desktop score is great but my mobile score is below 30 for over 500 pages :(
Edit: Another thing telling me Core Web Vitals is off — GSC says CLS is bad on 600 pages.
Thanks,
Jason- This reply was modified 3 years, 8 months ago by Jason. Reason: Add more data
Thanks Ismael,
When I test from the Pagespeed website the score is very different. I always assumed this was Chrome not being throttled.
I tried swapping to the Enfold gallery and it is marginally better. I also have the problem where all media on the page (eg. youtube videos in footer) appear in the gallery for the main product image. How can they be filtered?
With the Woocommerce gallery, the main image is not lazyloaded, only the thumbnails which should be below the fold. They appear above the fold due to the description tab taking so long to load.
Is there a way to make the description tab display immediately on pageload instead of it waiting on whatever event is drawing the tabs & +/- buttons currently?
Jason
Thanks Ismael,
Would appreciate you putting the screenshot in private links.
I don’t think your score in Chrome on a fast PC is indicative of a mobile device on a real world connection. Google use a throttled test when doing their scoring for Mobile, this is documented in many analysis’ of Core Web Vitals.
If you use Pagespeed Insights on the same page you see a score of 30 overall for Mobile. And maybe you are right, CLS is not the issue, it’s FCP.
Are you able to look at these metrics and provide some performance fixes for the theme? Google have been signalling very clearly that Core Web Vitals is going to become a ranking signal in 2021.
Thanks,
JasonYou’ll need to use Chrome Lighthouse, or the 3G throttling mode to see what I am referring to. On that page or related ones, with a 3G throttle:
– logo loads after menu
– all the related product thumbnails load before the main image (because they are above the fold due to description tag not yet expanding)
– +/- buttons for quantity load
– description tab loads pushing thumbnails below the fold
– then main product image loadsThe core web vitals scores in Lighthouse demonstrate this with very different scores compared to an unthrottled test like GTMetrix.
Thanks,
JasonFebruary 8, 2021 at 9:28 am in reply to: Enfold LayerSlider loading in footer causing high CLS #1278578Thanks Ismael,
The first two did not work. The last one did, and improved the load time of the above the fold slider.
Is it possible for you to include this option in the theme performance settings so we don’t have to resort to a code change? Or provide another snippet for the functions.php.
Thanks,
JasonFebruary 5, 2021 at 12:03 pm in reply to: Enfold LayerSlider loading in footer causing high CLS #1277954Merge & Compress.
Load only used elements.
Enable lazy loading.I don’t think any of these have an impact on layerslider?
Link below.
Cheers,
JasonLink below.
March 13, 2020 at 1:13 pm in reply to: 4.7.3 JS Merge & Compression aviaAsset_avia-head-scripts error-generating-file #1192872Yigit,
I have stated above what I believe to be the cause – having no scripts in the head, causing the script generation error – compounded with the timestamp on the options entry.
Your suggested fix still results in the error being generated when it shouldn’t be. It is a workaround, not an actual solution. The optimizer will try to generate a new head script every page load even though there is nothing to include. This is bad for performance.
I am a bit gobsmacked that you guys haven’t released an update to resolve this as there must be thousands of sites running Enfold with this issue. It’s nothing to do with my server, or caching.
Cheers,
JasonMarch 10, 2020 at 6:57 am in reply to: 4.7.3 JS Merge & Compression aviaAsset_avia-head-scripts error-generating-file #1191726I am not using WP Engine nor am I doing any database caching.
The fix in that thread does not work. Please re-read my message where I have given specific data as to what the cause is.
- This reply was modified 4 years, 8 months ago by Jason.
Seems this is related to https://kriesi.at/support/topic/4-7-3-js-merge-compression-aviaasset_avia-head-scripts-error-generating-file/#post-1191381
This new timestamp code is flakey.
March 9, 2020 at 6:23 am in reply to: 4.7.3 JS Merge & Compression aviaAsset_avia-head-scripts error-generating-file #1191381PS: my workaround was to edit line 507 of /themes/enfold/config-templatebuilder/avia-template-builder/php/asset-manager.class.php to
$content = ";";
This makes a dummy file so the routine does not spew errors on every page load.
- This reply was modified 4 years, 8 months ago by Jason.
March 8, 2020 at 2:03 pm in reply to: Cleanup wp_options Table – aviaAsset_avia-head-scripts #1191261I have been seeing the same issue since 4.7.3.
Were there changes in the caching algorithm?
- This reply was modified 4 years, 8 months ago by Jason.
I am running 4.7.3. Have you tried to replicate the issue?I thought I was on 4.7.3 but the upgrade failed. After upgrading to 4.7.3 the error is gone. Thanks!
- This reply was modified 4 years, 8 months ago by Jason. Reason: PICNIC
The fix you described above in post #1165405 is not included.
All you need to do is execute any wp-cli command and you will see errors relating to line 190 of the file mentioned in that post.
admin@:/path$ wp plugin status PHP Warning: Illegal string offset 'product_gallery' in /path/wp-content/themes/enfold/config-woocommerce/config.php on line 190 Warning: Illegal string offset 'product_gallery' in /path/wp-content/themes/enfold/config-woocommerce/config.php on line 190 PHP Warning: Cannot assign an empty string to a string offset in /path/wp-content/themes/enfold/config-woocommerce/config.php on line 190 Warning: Cannot assign an empty string to a string offset in /path/wp-content/themes/enfold/config-woocommerce/config.php on line 190 PHP Warning: Illegal string offset 'product_gallery' in /path/wp-content/themes/enfold/config-woocommerce/config.php on line 193 Warning: Illegal string offset 'product_gallery' in /path/wp-content/themes/enfold/config-woocommerce/config.php on line 193
February 23, 2020 at 11:55 pm in reply to: Enfold Instagram Cache + WP Super Cache causes 404s + Wordfence causes Bans #1187145The next release came out (January 21th – Version 4.7.2) and the fix is not included.
Can you please give us a version to expect it in?
This bug throws a bunch of warnings with every use of the wp-cli.
- This reply was modified 4 years, 9 months ago by Jason.
November 9, 2019 at 1:09 am in reply to: Enfold removes WooCommerce Product & Category Breadcrumb LD+JSON #1155131Again, I said above, this causes double breadcrumb to be inserted.
I am asking for a permanent fix to the plugin. Today Breadcrumb schema is critical for SEO. Enfold should work out of the box.
Jason
November 2, 2019 at 12:11 am in reply to: Enfold removes WooCommerce Product & Category Breadcrumb LD+JSON #1153159I don’t want to disable the theme breadcrumb.
I am suggesting that the theme should not break the Woocommerce JSON-LD breadcrumb.
October 29, 2019 at 11:46 pm in reply to: Enfold removes WooCommerce Product & Category Breadcrumb LD+JSON #1152179Check your own demo – https://kriesi.at/themes/enfold-shop/product/casual-collection/
There should be a breadcrumb json-ld tag inserted by WooCommerce.
Cheers,
JasonOctober 26, 2019 at 9:52 am in reply to: Enfold removes WooCommerce Product & Category Breadcrumb LD+JSON #1151326I do not allow remote access to my website for privacy and security reasons.
You should be able to reproduce this on a stock install of wooocommerce/enfold.
October 21, 2019 at 11:25 pm in reply to: Enfold removes WooCommerce Product & Category Breadcrumb LD+JSON #1150000Sorry but that link does not have information to resolve this problem, and I’m asking for a permanent fix. Due to the recent increase in importance of breadcrumb microdata (it’s now used in serps) thus should be a no brainer?
Jason
October 16, 2019 at 9:56 am in reply to: Enfold removes WooCommerce Product & Category Breadcrumb LD+JSON #1148356Hey Basilis,
There are two things at play here:
– the breadcrumb
– the breadcrumb schema data injected by WoocommerceBecause Enfold disables the stock Woocommerce breadcrumb to insert it’s own, it also inadvertently disables the schema data. If I use the snippet above I get the breadcrumb data but two breadcrumbs on the page.
Hope this clarifies the problem.
JasonOctober 13, 2019 at 12:01 pm in reply to: Enfold removes WooCommerce Product & Category Breadcrumb LD+JSON #1147569Thanks Victoria,
This adds a duplicate breadcrumb in the wrong place.
Is it possible to get a proper fix where we get the breadcrumb schema and a single correct breadcrumb?
Due to Google pushing breadcrumb schema this is going to become a hot SEO issue.
Thanks,
JasonOctober 7, 2019 at 6:49 am in reply to: Error in structure data (Error: SomeProducts is not a known valid target type) #1145641Is anyone able to confirm whether this fix will be included in a release of Enfold? I am seeing the same Structured Data Testing Tool errors.
Thanks,
JasonThat’s correct. I am on 4.6.2. Still broken.
Yes that helps. Is there a chance you can include the fix in the next version? At least make it optional.
Cheers,
JasonJuly 5, 2019 at 12:47 am in reply to: Merge and Compress CSS & JS Issue – avia_head_scripts #1115942I understand why you need to verify the compressed file is available, but your check should not mandate a signed cert; ie. you should allow self-signed certs when doing this check.
Cheers,
JasonThe constant error seems a little odd, but thanks for the response!
I hope the other thread gets a response though as the merge and compress is not working on my staging site due to the SSL cert.
Cheers,
Jason -
AuthorPosts