Navigation
This is archived content. Visit our new forum.

Tagged: 

  • Author
    Posts
  • #1292482
    hkf_22
    Participant

    I had a plugin for minifying but I deleted it. How come the code is still minified?

    #1292551
    Paul R
    Moderator

    Hi,

    That’s because the css and js code of the theme are already minified by default.

    No need to minify it again using a plugin or else it will cause an error.

    Hope that makes sense.

    #1292557
    hkf_22
    Participant

    Yes I understood. So what do I need to do now to make the slider always available.

    #1292772
    Lely
    Moderator

    Hi There,

    When I check again the error is gone now. I can view the slider from your homepage on desktop and on mobile too.
    Lightbox on your collection page are working too. See attached screenshot.

    #1293783
    hkf_22
    Participant

    Yes, on the desktop view it appears fine to me but when I view the collection page from my mobile, the page keeps reloading because of the error I mentioned. Also, I was just viewing my homepage again, and the revolution slider is duplicated and one of them is above the navigation bar.

    #1293870
    Rad
    Moderator

    Hi there,

    Sorry for the confusion, the minified content that we’re referring is the script embedded within HTML source code of your page. The error originates from here

    <script> jQuery(document).ready(function($) { var windowObj = $(window); var body = $('body'); var bodyOffsetBottom = windowObj.scrollBottom(); // 1 var bodyHeightAdjustment = body.height() - bodyOffsetBottom; // 2 var bodyHeightAdjusted = body.height() - bodyHeightAdjustment; // 3 var scrollTopAnchor = $('.x-scroll-top'); function sizingUpdate(){ var bodyOffsetTop = windowObj.scrollTop(); if ( bodyOffsetTop > ( bodyHeightAdjusted * 0.01 ) ) { scrollTopAnchor.addClass('in'); } else { scrollTopAnchor.removeClass('in'); } } windowObj.bind('scroll', sizingUpdate).resize(sizingUpdate); sizingUpdate(); scrollTopAnchor.click(function(){ $('html,body').animate({ scrollTop: 0 }, 850, 'easeInOutExpo'); return false; }); }); </script>

    And it’s broken, as you can see, it minified the code and made them in the single line. Hence the comment ( // ) became one line too and the code beside it are now covered by the comment block, making the code invalid. If you’re going to minify a content, make sure comments are deleted (this is automatic on other minifier, not sure about yours)

    Maybe you have disabled the minification, but not the cache. Please disable the entire W3 Total cache plugin and test it again. I also checked the W3 Total Cache Minifier and it is still enabled.

    Thanks!

    #1293899
    hkf_22
    Participant

    I just disabled the minifier in W3 Total Cache but I don’t see any changes.

    #1293932
    Rue Nel
    Moderator

    Hello There,

    Aside from W3 Total Cache do you have any other caching plugin? When I checked some of your pages, I noticed this:

    Please clear all your caching plugin’s cache before testing your site. I would also recommend that you use private browsing mode or use Chrome’s incognito mode and test your site again.

    Please let us know how it goes.

    #1297206
    hkf_22
    Participant

    I deactivated all the plugins except for the revolution slider and it’s still as it is. Why are there two sliders at the top of the header and another one where I actually want it to be?

    #1297445
    Rad
    Moderator

    Hi there,

    You’re right, seems like the functionality weren’t attached or removed from the mobile menu. But checking this http://www.hayakhalifa.com/hk/wp-content/themes/x/framework/js/dist/site/x-head.min.js shows the code responsible for the mobile menu is there.

    Would you mind providing your FTP login credentials? And where is this hosted?

    Thanks!

    #1303834
    hkf_22
    Participant
    This reply has been marked as private.
    #1304093
    Rad
    Moderator

    Hi there,

    It’s now working but on different PC where I tested it. It looks like related to cache, could you try it again?

    Thanks!

    #1304859
    hkf_22
    Participant

    Unfortunately it’s still not working properly…

    #1305173
    Rad
    Moderator

    Hi there,

    I checked again and it still works. Perhaps we’re referring to different issues?

    Please provide a video recording, the screenshot fails to attach this time.

    Thanks!

    #1305315
    hkf_22
    Participant

    Please see the picture in the link below, you will notice the revolution slider is being duplicated, one before the navigation bar and one after it (where I actually want it to be)

    http://prnt.sc/dobs1h

  • <script> jQuery(function($){ $("#no-reply-1289584 .bbp-template-notice, .bbp-no-topic .bbp-template-notice").removeClass('bbp-template-notice'); }); </script>