First Item Soft rejected. Spacing Issues and Strick mode. Help needed

Hi…!! My One page template got soft rejected. I solved all other issues but i am confused in below two.
Here is the live link of my template
https://alloutdev.bitbucket.io/react-portfolio-template/
The reviewer has given screen shot of About section. I have used bootstrap grid there with 5, 3 and 4 columns respectively. Spacing seems fine to me. should i make all 3 sections even with 4,4,4 grid?
2. SPACING ISSUES: Many of the elements in the template are not aesthetically spaced.

  1. STRICT MODE REQUIRED: All JavaScript should be written with “use strict” mode on. Please note that strict mode is scoped. For concise code it is recommended to placed on top of the Javascript file.
    https://alloutdev.bitbucket.io/react-portfolio-template/

Take a look here

And this is my opinion and should be need improved.

Hope this help you

Thanks for pointing out clearly

Hi…!!! Braveart3D.
I again got soft rejection. This i got only one reason.

  1. STRICT EQUALITY COMPARISON: For better performance please use Strict Equality Comparison.
    Can you please take a look on my js file and tell me where the problem is. JS is very little.
    “use strict”;
    $(document).ready(function() {
    // MixItUp Start
    mixitup(’#mix-wrapper’, {
    controls: {
    toggleLogic: ‘and’
    },
    animation: {
    effects: ‘fade rotateZ(-180deg)’,
    duration: 700
    },
    classNames: {
    block: ‘programs’,
    elementFilter: ‘btn’,
    elementSort: ‘sort-btn’
    },
    selectors: {
    target: ‘.mix-target’,
    control: ‘[data-mixitup-control]’
    },
    });
    // MixItUp End
    // Smooth Scroll Start
    var navInneer = $(".scroll-nav");
    navInneer.singlePageNav({
    updateHash: false,
    filter: “:not(.external)”,
    offset: 61,
    speed: 1000,
    currentClass: “sdm-active”,
    easing: “swing”
    });
    $(".scroll-nav li a[href^=’#’]").on(‘click’, function(e) {
    e.preventDefault();
    $(’.scroll-nav li a’).removeClass(‘sdm-active’);
    $(this).addClass(‘sdm-active’);
    });
    // Smooth Scroll End
    // Add header class on scroll Start
    var win = $(window);
    win.on(‘scroll’,function() {
    var scroll = win.scrollTop();
    if (scroll >= 400) {
    $(".scroll-nav").addClass(“scroll-down”);
    } else {
    $(".scroll-nav").removeClass(“scroll-down”);
    }
    });
    // Add header class on scroll End
    // preloader
    win.on(‘load’, function() {
    $(’.sk-wave’).fadeOut();
    $(’#preloader’).delay(350).fadeOut(‘slow’);
    });
    // preload end
    });

I think the reviewer is pointing to this line if (scroll >= 400) { but i am not sure.

After document ready !

$(document).ready(function() {
"use strict";
1 Like

Thanks for you help item got approved.
https://themeforest.net/item/react-one-page-portfolio-responsive-html-5-template-for-freelancers-agencies/19516254 :slight_smile:

2 Likes

Congrats for approved your template :slight_smile:

1 Like