Forum Replies Created

Viewing 30 posts - 241 through 270 (of 426 total)
  • Author
    Posts
  • in reply to: WordPress 5.8 Widget Blocks Error #1311489

    @NIXHALBES

    Just looked at the plugin source code LOL yep definitely will be entering the filters manually into functions.php! Thanks for sharing this one!

    The plugin is literally two lines of code with a load of annotations and a readme file.

    in reply to: WordPress 5.8 Widget Blocks Error #1311488

    Cheers @NIXHALBES
    If Kriesi and team are not planning on integrating Gutenberg Footers then the code above should be considered for the core theme to avoid confusion.

    I’ve not tested the code yet but looks right to me.

    in reply to: Logo SrcSet? #1310913

    @Yigit
    I have sent you three emails – Email 3 sums up the two issues with Srcset and a possible fix.

    In The Beta release for 4.8.4
    I modified $dimension on line 710 in function-set-avia-frontend.php so the dimension matches the uploaded size of the image.

    $dimension = apply_filters( ‘avf_logo_dimension’, ‘height=”116″ width=”450″‘ );

    This removes the warning from Chrome.

    All ALB image elements state the height and width automatically based on the chosen image size. This logic needs to be applied to the logo.

    The size of the logo is about 68% the width of the viewport. So to get it to serve the right size image for the space I think the size attriute on the logo should be:
    sizes=”(max-width: 450px) 68vw, 450px”
    NOT
    sizes=”(max-width: 450px) 100vw, 450px”

    Hope this helps.

    • This reply was modified 3 years, 6 months ago by thinkjarvis.
    in reply to: Bloated database? #1310434

    A link to your website will allow the support team to see what plugins and setup you have.

    They can’t help you without it.

    Some plugins create their own tables in the database which can bloat the database. Sometimes the way post data is stored can also increase the size of the database.

    in reply to: Enfold Showcase #1310370

    @kommo
    Thanks. Quite a bit went in to this one from an optimisation perspective for looks and speed.

    The above the fold section is an ALB color section with background image. I have set a minimum height depending on the image requirements for each screen size (Either 350px or 600px). I then applied a custom class and custom ID to each to set the height depending on the screen size. This removes issued with CLS in Google Web Vitals because the sizes are explicitly stated early.
    If you view source in Google Chrome you can see the code added as critical CSS into the header. I used wp_head to add code directly into the header from functions.php.

    Most of the mods come from a performance perspective:
    The logo and srcset images are pre-loaded so the logo is available as early as possible.
    Self hosting fonts and Adding font preload manually from functions.php just like the other critical CSS. This prevents Font Swap (Flicker) from occuring as the fonts are available right away.

    To improve LCP and FCP scores you can either pre-load the background color section image in the same way as the fonts so it is available right away or add critical css in the way that I have – serving a different image size per screen size.

    I use critical CSS on Burden Basket because of the minimum height applied to the ALB colour section.

    To ensure the images look right on all screen sizes I have used media queries to load a different banner image per screen size on every page.

    I have enfold setup to have mobile break points at 989px instead of 767px. I do this in the Enfold control panel and every column element on every page .

    As the page scales down (if you resize the browser window) I have used media queries to change the menu font size and padding so that it works on screens all the way down to 989px. You can adjust the mobile breakpoint if required to lets say 1024px (for ipads) with some custom CSS. If you search the forum you’ll find this one.

    Although most sites read Left to Right from a mobile design perspective – Put the first image on the left – This will mean users see picture content first in the same way people are familliar with on social media. Its good UX design to do it this way but looks a little unusual on desktops. You could always set two images and hide the first one on desktops if you prefer them the other way around on different platforms.

    I hope this helps.

    in reply to: Enfold Showcase #1310093

    @justin
    The menu design for Burden Basket used a combination of features from the Enfold Documentation.

    Sorry for the late reply.
    I used a combination of settings from the Enfold Documentation.
    The logo layout is set to Logo Center, Menu Below
    The logo is then re-positioned to the left using Quick CSS

    Custom Widget added to header

    The widget is then re-positioned to the right with Quick CSS

    Bonus Points
    If you add a search widget into the customer widget area – the Ajax dropdowns still work because it is within the header area. Great for Ecommerce sites.

    Justified Menu (equal spaced menu)

    Colours and menu break point adjusted with Quick CSS.

    Hope this helps. It took a bit of fiddling but I have a simillar design running on most of my ecommerce sites becuase it makes them look and function more like a typical ecommerce site.

    in reply to: Bloated database? #1310091

    Unoffocial response:
    Have you tried using something like WP-Optimize to clean the database?

    Please use with caution. It will delete all post revisions and posts from the bin and spam folders.
    I use this with every website I design. Helps lean up the database and keep things running smoothly.

    250 posts is quite a lot. I have a couple of Ecommerce sites that have breached a 200mb database but these have a few thousand products.

    in reply to: New Google Track codes not compatible with Cookie Modal #1310090

    Hi ismael,
    Sorry yes the script is working but the cookie modal was not. When a new G4 code is entered into the Google services box, it no longer provides the option to turn off the cookie in the cookie modal.

    I have just managed to get it to display and function by doing the following:
    I had to turn on the custom Modal Window Custom Content setting.
    Then save. This makes the Google Analyitcs box appear in the Cookie Modal again.

    Not sure why it wasn’t working.

    Thanks anyway you can close this issue.

    in reply to: LCP issue caused by fade-in on first element #1310049

    THP thanks for the suggestion.

    If the text is the LCP element try serving your fonts locally and adding preload to the header file (you can do this from functions.php). This will help prevent CLS shift and improve LCP and FCP scores in most cases. If you download the fonts you can use cloud convert to convert a .ttf font to a .woff2 which is much smaller.

    I think one of the issues is that when CSS is merged into one file, the page layout is unavailable until this loads – Meaning that any calculated image sizes like the logo with Srcset wont render until the browser gets the size info from the CSS file. The only way around this is to generate critical CSS.

    I have tried Jetpack Boost to add critical CSS (new plugin from the official WordPress team). This should fix the problem but I need to experiment more with it yet. Ironically I also tried inlining all of the CSS so there were NO CSS FILES and this actually reduced LCP to a pass score of under 2.5 seconds! But not best practice given the 100kb of CSS in the theme! The TTI score got a lot worse!

    Try using Jetpack Boost if you are not already using a plugin that generates critical CSS.

    Regarding Doodles Of Art – When you open Chrome it will always load in desktop mode. Even when using dev tools. So it will already have the larger image saved in cache when using dev tools to change screen sizes. If you run Lighthouse in mobile mode – It does render a smaller logo image.

    Will wait to see if team Kriesi have any advice on the LCP issue – specifically regarding the logo as the LCP.

    • This reply was modified 3 years, 6 months ago by thinkjarvis.
    in reply to: Problem with google analytics #1310048

    Possibly!

    In WP Rocket try adding analytics.js and gtag.js to the exclusion list in the JS settings.
    Also add jquery.min.js if you run into any other problems.

    If you defer all inline JS you may be accidently deferring Enfolds Cookie Consent Modal which normally loads in the <head> section of each page. This will probably stop it working properly.
    This may also be defering the track code.

    Hope this helps :)

    • This reply was modified 3 years, 6 months ago by thinkjarvis.
    in reply to: Problem with google analytics #1309960

    Hi Ismael,

    I have just raised an issue regarding the New G4 track codes when i saw this post. In 4.8.3 release the new G4 track codes break the Cookie Consent Modal (this has happened before with a previous Enfold release).

    When a new G4 tag is added to Enfold in the tracking code box, Cookie control does not work and the script gets loaded anyway.

    I can still see data in the Analytics console though but there is definitely a bug with the latesg G-XXXXX tags.
    My post is here if it helps:

    I tried the same sites with an old UA-XXXX code and it works OK. So I can see a problem with new G4 G-XXXXXX codes.

    in reply to: Logo SrcSet? #1309955

    Cheers Gunter,

    For your notes:
    The current 4.8.3 implimentation of Srcset does not explicitly give the logo element a height and width tag. This flags an error when testing with Lighthouse or Page Speed Insights.

    I have had to manually add rules for different screen sizes to add width and height tags to the logos. You can see this in the private text below or in the <head> as critical css on the example below:

    in reply to: LCP issue caused by fade-in on first element #1309933

    I have just been looking at the performance/network panel in Chrome Lighthouse.

    For doodles-of-art.co.uk and ashbybasketball.co.uk after the initial render, the logo then disappears and re-appears at the LCP point. After JS has loaded an animation is triggered and the LCP is recorded as after the animation finishes.

    The LOGO recieves a fade in animation in both these examples.

    This might be the cause of the issue because the logo does not animate on the burdenbasket.co.uk.

    Is there anything I can do to prevent this behaviour?

    I’m keen to hear your suggestions.

    • This reply was modified 3 years, 6 months ago by thinkjarvis.
    in reply to: Enfold Showcase #1307820

    Another new site launch.
    Burden Basket (HOPE Project) is a mental health support and training provider for children and young people. Alongside therapeutic counselling services, the HOPE project provides both training and support services to teachers and children. Based in Swadlincote, UK – Covering South Derbyshire, Staffordshire and Leicestershire.

    https://www.burdenbasket.co.uk

    Working with the company owners, the website content has been updated to contain more information about the services offered – Missing from the old website.
    The site uses a custom header design and menu – Full width and equally spaced with a custom widget area for contact information.
    Mobile optimised header images have been added manually by replacing the desktop sized ALB Colour Section Background image with correctly sized versions for different screen sizes.
    This code is then inserted into the header via functions.php as inline critical CSS to improve FCP and LCP scores.

    A calendar plugin has been used to display Eventbrite and zoom calendar entries – embedded with loading=”lazy” tags for speed.

    The site passes all web vitals metrics at time of launch.

    • This reply was modified 3 years, 7 months ago by thinkjarvis.
    in reply to: Enfold Showcase #1307817

    Been a bit quiet on the social posts lately! Lots of work for existing customers but we have a couple of new launches this month:

    Holding page designed for a company selling CBD oils and topical creams (all legal in the UK as a food supplement if you were wondering). The company is based in the UK and provides some of the highest quality Pure CBD oil on the market.

    The holding page was set up specifically as a funnel for the companies main website (which I have not been able to persuade over to Enfold yet!). The layout has menus turned off.

    https://www.hemppureoils.co.uk

    Page speed is important for a landing page. The Hemp Pure Oils site scores 98/100 on mobiles on a good test run. However this varies depending on how well the server responds etc. LCP sits at 2.3 seconds on a good run but rises to 3.6 seconds on a bad run.

    Some critical CSS is loaded in WP Head to improve the FCP and aid the LCP but more is required for a bigger impact.

    The site uses free optimisation tools and does not use a CDN at this point in time showing just how good Enfold is as a platform for websites compared to others like DIVI and Elementor which all require more work to get a 90+ score in page speed insights.

    I hope this proves useful for anyone else looking to build a fast and easy to edit landing page.

    in reply to: Disable fade on first element #1307458

    I’ll give it a try and let you know.

    I am trying to resolve the LCP issues with Enfold at the moment. In some cases all is fine, in others it is not.

    Good comparison for you:

    https://www.burdenbasket.co.uk has an empty color section as the main header and the LCP score is still pretty good

    https://www.hemppureoils.co.uk as per previous posts has a color section with elements in it and the LCP score bounces around a lot more depending on how well the test executes.

    Both sites will score all green on occassion but with elements within the color section things slow down.

    I think the only solution is to add critical css inline for the entire above the fold section and remove client side rendering including animations on mobile devices.

    Another point probably overlooked by all Enfold users – If you follow ICO’s guidelines (GDPR) to the letter, you should only be loading essential cookies and anything marketing should be on opt-in only. Doing this stops analytics.js loading on pageload – Improving pagespeed scores by up to 10 points and ensuring GDPR compliance.

    in reply to: Disable fade on first element #1307343

    Just edited the response Opacity 0 means that the elements are not visible.

    Opacity set to 1 solves the load issue but adds a layout shift so I have set it back to the following:

    @media only screen and (max-width: 1024px) {
    .js_active .av-minimum-height .container {
    z-index: 1; /*required for transition*/
    opacity: 0;
    -webkit-transition: none !important;
    transition: none !important;
    }
    }
    • This reply was modified 3 years, 7 months ago by thinkjarvis.
    in reply to: Disable fade on first element #1307341

    Thanks for the quick response @yigit

    I added the following based on your suggestion as critical css to the header:
    Not entirely sure it has made a lot of difference but the site is JUST passing LCP with a score of 2.3 to 2.5 sseconds on a GOOD test run in Chrome Dev Tools.
    Total score is 98/100 on a good run.

    @media only screen and (max-width: 1024px) {
    .js_active .av-minimum-height .container {
    z-index: 1; /*required for transition*/
    opacity: 1 !important;
    -webkit-transition: none !important;
    transition: none !important;
    }
    }
    • This reply was modified 3 years, 7 months ago by thinkjarvis.
    in reply to: Fixing CLS errors #1305244

    Thanks @ismael

    Looks like your solution works but has affected the layout slightly.


    @sldeutsch
    site appears that the layerslider is served at the same height on all screen sizes excluding one.

    I just encountered a similar problem on a client site with an ALB background element that becomes 1400 pixels tall on mobiles.
    The layout shift was 2.48 then down to 0.99 with a min-height on the element within Enfold and then down to 0.01 with an explicitly stated height as per your solution.

    Setting min-height alone only works if the shift on mobiles is minimal so I added the following height:; css.

    This is a specific case where the elements stack. The LCP score is still a bit high but very hard to optimise because the design calls for such a large first element which stacks on mobiles.

    But you get the idea:
    -serve different background image for smaller/taller screens
    -set fixed height for those elements for different screen sizes

    
    /* Media Rules for ALB Colour Element Hero Images */
    
    /* mobiles  */
    @media screen and (max-width:767px) {
    #main-banner-1 {background-image: url(/wp-content/uploads/cbdbgd-wood-5-767-1500-1.jpg)!important;}
    }
    
    /* mobiles  */
    @media screen and (max-width:767px) {
    #main-banner-1 {
    min-height:1400px !important;
    height:1400px !important;
    }
    .tj-main-banner-style {
    min-height:1400px !important;
    height:1400px !important;
    }
    #main-banner-1 .container {
    min-height:1300px !important;
    height:1330px !important;
    }
    }
    
    /* Desktops  */
    @media screen and (min-width:768px) {
    #main-banner-1 {
    min-height:689px !important;
    height:689px !important;
    }
    .tj-main-banner-style {
    min-height:689px !important;
    height:689px !important;
    }
    #main-banner-1 .container {
    min-height:689px !important;
    height:689px !important;
    }
    }
    • This reply was modified 3 years, 7 months ago by thinkjarvis. Reason: Correction
    in reply to: Fixing CLS errors #1305111

    You have to go through the page from top to bottom to isolate the causes. Unfortunately I would need to quote you for the work if you want me to look at it.

    CLS issues and web vitals work is quite in depth.

    The largest shift is caused by the layer slider image. As suggested you have to explicitly state the height of the element to prevent the shift and should be done to match different device sizes. This can be done in CSS.

    Once the layerslider is no longer causing a shift you may find that the other components load far enough down the page that web vitals no longer considers them a problem.


    @ismael
    may still be able to help you as part of the support team.

    in reply to: Fixing CLS errors #1304593

    @ismael

    Sorry I have just realised I missed the closing brackets in my code snippet I have edited my response. Sorry if I have made the job harder for you!

    in reply to: Logo SrcSet? #1303790

    performance figures for https://www.midlandcontrols.co.uk

    Was 89/100 without Srcset
    Now 98/100 with Srcset (and my bespoke settings for Enfold)
    https://www.midlandcontrols.co.uk/wp-content/uploads/performance-figures-new-1.jpg

    in reply to: Logo SrcSet? #1303782

    Cheers Ismael,

    I can confirm that this works (I have implemented this on another website).

    in reply to: Logo SrcSet? #1303435

    In order to get this to work Unofficial workaround. Please backup the file before trying.

    Working example
    https://www.midlandcontrols.co.uk

    Set the logo in the theme panel to Full Size! Do not choose a thumbnail size!

    Delete this line 625 from function-set-frontend.php

    		if( $dimension === true ) 
    		{
    			/**
    			 * basically just for better page speed ranking :P
    			 * Be sure to return a valid attribute string.
    			 * Will be removed when image dimension is responsive
    			 * 
    			 * @since 4.8
    			 * @param string $dimensions
    			 * @return string
    			 */
    			$dimension = apply_filters( 'avf_logo_dimension', 'height="100" width="300"' );
    		}

    Delete the code below Line 684

    {
    				$resp_attr = Av_Responsive_Images()->html_attr_image_src( $logo, true );
    				
    				/**
    				 * https://kriesi.at/support/topic/logo-srcset/
    				 * 
    				 * Bug that WP removes scrset and sizes when width and/or height is defined
    				 * @since 4.8.3 
    				 */
    				if( false !== strpos( $resp_attr, 'srcset' ) || false !== strpos( $resp_attr, 'sizes' ) )
    				{
    					$dimension = '';
    				}
    			
    				$logo_img = "<img {$resp_attr} {$dimension} alt='{$alt}' title='{$title}' />";
    				$logo = "<{$headline_type} class='logo'><a href='{$link}'>{$logo_img}{$sub}</a></{$headline_type}>";
    			}

    Replace with:

    			{
    				$resp_attr = Av_Responsive_Images()->html_attr_image_src( $logo, true );
    			
    				$logo_img = "<img {$resp_attr} alt='{$alt}' title='{$title}' />";
    				$logo_img = Av_Responsive_Images()->make_image_responsive( $logo_img, attachment_url_to_postid($logo), '' );
    				$logo = "<{$headline_type} class='logo'><a href='{$link}'>{$logo_img}{$sub}</a></{$headline_type}>";
    			}
    • This reply was modified 3 years, 8 months ago by thinkjarvis.
    in reply to: Fixing CLS errors #1303375

    @Ismael is right with his solution.
    Start from the top of the page and work downwards. CLS shift tends to only apply on above the fold because the rest of the page loads out of view.

    If you can remove the shift from the first element on all devices then the rest may no longer be an issue. As they are all ALB colour sections, if you know that it will dynamically resize, set no minimum height. If it has a background image set then you may still experience some shift. You will need to create a version of the code below for that element.

    The Layerslider needs an explicitly stated height to prevent shift. ALB background elements and Sliders also have this issue. You essentially need a few rules to match the design:

    Below is suitable for the layerslider. HOWEVER you may need to apply it to the colour section and the background image as well to prevent the shift. I tend to set a custom ID so that the rule only applies to this element. See how you get on with the below:

    /* Desktops 1080p */
    @media screen and (min-width:1441px) {
    .home #layer_slider_1 .ls-wp-container, .home #layer_slider_1 .ls-wp-container .ls-inner {
        height: 271px !important;
    }
    }
    
    /* Macbooks, 720p and ipad pros  */
    @media screen and (min-width:1025px) and (max-width:1440px) {
    .home #layer_slider_1 .ls-wp-container, .home #layer_slider_1 .ls-wp-container .ls-inner {
        height: 272px !important;
    }
    }
    
    /* Ipads landscape */
    @media screen and (min-width:769px) and (max-width:1024px) {
    .home #layer_slider_1 .ls-wp-container, .home #layer_slider_1 .ls-wp-container .ls-inner {
        height: 271px !important;
    }
    }
    
    /* Ipads portrait */
    @media screen and (min-width:451px) and (max-width:768px) {
    .home #layer_slider_1 .ls-wp-container, .home #layer_slider_1 .ls-wp-container .ls-inner {
        height: 271px !important;
    }
    }
    
    /* mobiles portrait */
    @media screen and (min-width:361px) and (max-width:450px) {
    .home #layer_slider_1 .ls-wp-container, .home #layer_slider_1 .ls-wp-container .ls-inner {
        height: 321px !important;
    }
    }
    
    /* Moto G4 portrait */
    @media screen and (max-width:360px) {
    .home #layer_slider_1 .ls-wp-container, .home #layer_slider_1 .ls-wp-container .ls-inner {
        height: 271px !important;
    }
    }
    • This reply was modified 3 years, 7 months ago by thinkjarvis.
    in reply to: Logo SrcSet? #1303368

    Thanks THP,

    @ismael
    – Let us know when you have taken a look.

    in reply to: Logo SrcSet? #1303082

    Hi Ismael,

    I just replaced the logo on the example I sent over with a 2000 pixel wide image, checked via ftp to make sure image sizes were generated and set it as the logo.

    It just serves the original Srcset does not apply.

    The workaround for 4.8.2 you provided me worked fine in 4.8.2. But the new code added for 4.8.3 does not work and now the code has changed the workaround does not work either.

    Let me know if you have a fix for this.

    in reply to: Logo SrcSet? #1302786

    Hi Ismael,

    Just updated to 4.8.3. Srcset logos still not included.

    Sorry to report bad news!

    I was going to reinstate the mod but some of the other code changes are preventing it working.
    Please advise.

    • This reply was modified 3 years, 8 months ago by thinkjarvis.
    in reply to: Absolutely bad speed values #1300681

    What kind of server are you using apache? or NGINX?

    Most NGINX servers prevent the built in script merge loading. I have experienced this with WPEngine hosted sites. They seem to ignore the script merge when it is enabled (even after the cache is cleared with the CDN disabled).

    Third party plugins like WP-Rocket will still work. However the server will likely be http2.0 meaning that script merging is not so crucial.

    Try using Google Chromes built in Lighthouse testing set to mobile with throttling turned on. It gives better ‘lab’ conditions and because it is browser based it can work on test servers.
    You can then cross reference it against GT Metrix, Page Speed Insights and WebPageTest.org

    in reply to: Logo SrcSet? #1297502

    Ive tested it. The code above works just fine.

    This problem appear to apply to all Enfold sites and will need addressing in the next release.

Viewing 30 posts - 241 through 270 (of 426 total)