Прокрутка сверху вниз не работает

Question

У меня проблема с прокруткой, 2 из if условия активируются все время после некоторой задержки. Я не могу понять почему. Вот сайт, с которым у меня проблема: http://fitfaza.pl/ - серый якорь в правом нижнем углу.

код:

// hide #back-top first
jQuery("#back-top").hide();
jQuery("#back-top2").hide();
// fade in #back-top
jQuery(function() {
    jQuery(window).scroll(function() {
        if (jQuery(this).scrollTop() > 500) {
            jQuery('#back-top').fadeIn();
            jQuery('#back-top2').fadeIn();
        } else {
            jQuery('#back-top').fadeOut();
            jQuery('#back-top2').fadeOut();
        }
    });


    jQuery('#back-top a').click(function() {

        jQuery('body,html').animate({
            scrollTop: 0
        }, 800);
        return false;
    });

    $(window).scroll(function() {
        var scroll = $(window).scrollTop();
        if (scroll <= 1089) {

            jQuery('#back-top2 a').click(function() {

                jQuery('body,html').animate({
                    scrollTop: 3390
                }, 800);
                return false;
            });
            $("html, body").bind("scroll mousedown DOMMouseScroll mousewheel keyup", function() {
                $('html, body').stop();
            });
        };
        if (scroll >= 3390) {

            jQuery('#back-top2 a').click(function() {

                jQuery('body,html').animate({
                    scrollTop: 1
                }, 800);
                return false;
            });
            $("html, body").bind("scroll mousedown DOMMouseScroll mousewheel keyup", function() {
                $('html, body').stop();
            });
        };
    });

});

Answer 1

Вот оптимизированный скрипт, который, кажется, имеет свой эффект - хотя я не могу хорошо проверить реальный ответ на сайте, не удаляя текущий активный скрипт с легкостью. Поэтому вам, возможно, придется протестировать его и дать некоторую обратную связь о его функциональности. В любом случае, попробуйте заменить это:

http://codepen.io/anon/pen/pvMmyJ?editors=001

$(function () {

  var gate = $(window);
  var root = $('html, body');
  var backtop = $('#back-top2');
  var scroll;
  backtop.hide();

  gate.scroll(function() {

    scroll = gate.scrollTop();

    if (scroll >= 500 && !backtop.is(':visible')) backtop.fadeIn();
    if (scroll < 500 && backtop.is(':visible')) backtop.fadeOut();
  });

  backtop.click(function() {
    if (scroll < 1090) root.animate({scrollTop: 3390}, 800);
    if (scroll >= 3390) root.animate({scrollTop: 0}, 800);
  });

  root.on('scroll mousedown touchstart DOMMouseScroll mousewheel keyup', function() {
    root.stop();
  });

});

Глядя на страницу источника сайта, <a> метки вокруг якорного изображения не являются действительно необходимыми. И, похоже, $ не является зарезервированным символом, поэтому нет необходимости использовать jQuery для объекта. Кэширование некоторых переменных, которые используются повторно, также немного оптимизирует скрипт.