Forum Replies Created

Viewing 5 replies - 1 through 5 (of 5 total)
  • Thread Starter kmbt

    (@kmbt)

    @kbat82, the website in question is www.bib.olsztyn.pl. You can easily spot the files by looking for “301” in the Status column in the Chrome Developer tools, “Network” tab. If you have any additional questions or requests concerning the issue please let me know. I will try to answer them best as I can.

    Thread Starter kmbt

    (@kmbt)

    Hi, @kbat82 ,

    I’m sorry for having missed your reply. Thank you for your time.

    The font is actually queried by the browser causing unnecessary network queries. You can see that for yourself in the Chrome browser developer tools, “Network” tab. I am concerned about this because of unnecessary network load it may cause for mobile users.

    I have discovered this by accident when migrating a WordPress website from HTTP to HTTPS. The website has been using a plugin that redirects all queries that would normally return 404 error, to the home page (by means of 301 HTTP status code). The redirection plugin had been still configured for the https:// address and so the browser displayed a mixed content warning for the font files.

    I believe that removing the references to the font files from CSS should do no harm as they files are not present anyway. What’s more, this would improve mobile network performance by a tiny bit.

    • This reply was modified 6 years, 11 months ago by kmbt.
    • This reply was modified 6 years, 11 months ago by kmbt.
    Thread Starter kmbt

    (@kmbt)

    Oops, the forum script did not escape my nbsps ??
    It should be:

    In line 178 we have:
    $content = str_replace(‘ ‘,”,$content);

    And the two proposed solutions should stand:
    1. Do not remove nbsps
    2. Replace nbsp with a normal space.

    Hi, Roel

    Your fix still does not take into account one thing: On some displays the counter might be visible from the start, whereas on others it might require scrolling the counter into viewport. Maybe you should consider the following code:

    
        function wpcjs_check_visibility() {
    			var currentPosition = $( document ).scrollTop();
    
    			for ( var i in counterObjects ) {
    				if ( currentPosition > counterObjects[i].objectPositionTop && eventFired === false ) {
    					counterObjects[i].numAnimObject.start();
    				}
    			}
    
        }
    
    	window.onload = function() {
    		wpcjs_get_counter();
    		wpcjs_check_visibility();
    		$( window ).on( 'scroll', wpcjs_check_visibility);
    		
    	};
    

    This way, the loop checking positions of object is fired no only on Scroll, but also on Load event.

    This should be:

    .crellyslider > .cs-preloader { background-color: transparent; }

    as as none is not a valid background-color value. Please take look at the following Stack Overflow discussion: https://stackoverflow.com/questions/8739665/is-background-colornone-valid-css

Viewing 5 replies - 1 through 5 (of 5 total)