Viewing 11 posts - 1 through 11 (of 11 total)
  • Author
    Posts
  • #1220914

    Hi,

    Since 4.5.7, a potential infinite loop that breaks wp-admin in config-templatebuilder/avia-template-builder/php/popup-templates.class.php
    You must add a check to verify that $elements does indeed contain something. E.G.:

    public function replace_templates( array $elements )
        {
          $start_check = !empty($elements);
          ...

    Since 4.7.3, in config-templatebuilder/avia-shortcodes/slideshow/slideshow.js, use the correct variable (key instead of i), around line 158, in the for loop E.G.:

    	for( var i in htmlData )
    	{
    		//	data attribute is transformed to lower case, but js is case sensitive - transform key
    		var key = ( 'transition_speed' != i ) ? i :'transitionSpeed';
    		
    		// 4.7.4.1: Günter - in my opinion we should check newOptions[key], this is always true.
    		if( htmlData.hasOwnProperty( key ) )
    		{	
    			if( typeof htmlData[key] === "string" || typeof htmlData[key] === "number" || typeof htmlData[key] === "boolean" )
    			{
    				newOptions[key] = htmlData[key];
    			}
    		}
    	}

    Since 4.7.4.1, this margin-top added in css/grid.css, that serves no purpose except messing up with existing styles :

    #top .flex_column_table{
    ...
    	margin-top: 50px; /* please remove this margin */
    ...
    }
    • This topic was modified 4 years, 5 months ago by grizzlydev.
    #1221202

    Could you please also add something to override these in config-woocommerce/config.php

    #
    # removes the default post image from shop overview pages and replaces it with this image
    #
    add_action( 'woocommerce_before_shop_loop_item_title', 'avia_woocommerce_thumbnail', 10);
    remove_action( 'woocommerce_before_shop_loop_item_title', 'woocommerce_template_loop_product_thumbnail', 10);
    #1221220

    Hey grizzlydev,

    Thanks for reporting this. Will be fixed in next update.

    Enjoy the theme and have a nice day.

    Best regards,
    Günter

    #1221227

    sorry i try again.

    • This reply was modified 4 years, 5 months ago by Guenni007.
    #1221231

    Well, the thing is that as the code is in 4.7.5, the slideshow doesn’t transition at all and the controls don’t respond to clicks. No js errors, just a stuck slideshow.

    #1221237

    i tested it again :
    there was the topic of mine here: https://kriesi.at/support/topic/transition-speed-in-slider-alb/
    that on fading the images – the fade effect from last to first image on looping was with its default transition speed.
    so there was a fix for slideshow.js on : https://kriesi.at/support/topic/transition-speed-in-slider-alb/#post-1219428
    that was not perfect but much better than before.
    But when inserting your changings in that fixed file – the fading from last to first image is as before : wrong.

    #1221238

    Hi,

    Please try to use this file (check line 156ff):

    https://github.com/KriesiMedia/enfold-library/blob/master/temp_fixes/Enfold_4_7_5/alb-slideshow/slideshow.js

    Best regards,
    Günter

    #1221245

    yes – Günter that works as the file from before. with f.e. 7sec transition speed
    ( with that fix on line 838ff for fading last to next )
    https://webers-testseite.de/pureinstall/fullwidth-slider/

    but

    use the correct variable (key instead of i)

    when inserting this it is gone – even the transition speed goes back to default. not to the custom value.
    So that above from grizzlydev could not be the whole truth

    #1221246

    I don’t know what to tell you. I can’t see the fix you mention : it’s in a private field.
    It’s possible that the fixes I propose in the first post are not perfect. Works for me though.
    (Precision : the slideshow problem occured with an “Easy Slider”).

    #1221247

    All sliders go back to that js file !

    For me it is working before with the 4.7.5. the only little bug was on transition speed from last to first image on looping and fading (not sliding)
    As you can see on Günters link to github that was solved by additional code on lines 838ff:

    if (self.current === 0) {
    	hideSlide.avia_animate({opacity:0}, self.options.transitionSpeed/2, 'linear');
    	displaySlide.css(properties).avia_animate({opacity:1}, self.options.transitionSpeed/2, 'linear',fadeCallback);
    }
    #1222735

    Hi,


    @Guenni007

    The link to Github I provided above should work – but there is a problem with the fading effect I can reproduce – but I have not found a solution for it yet. I forwarded it to team member who has more experience with animations.

    Best regards,
    Günter

Viewing 11 posts - 1 through 11 (of 11 total)
  • You must be logged in to reply to this topic.