ブラウザ幅でのjQueryの切り替えについて

Question

前提・実現したいこと

レスポンシブのサイトを作成しています。
1024px以下では、「.gHeadBtnArea」がスクロールすると表示されて、1025px以上では、何も起こらないようにしたいです。

発生している問題・エラーメッセージ

1024px以下にブラウザ幅を小さくしてから、1025px以上にしたとき、スクロールすると「.gHeadBtnArea」が一瞬消えてしまいます。

該当のソースコード

$(window).resize(function(){
var w = window.innerWidth;
var x = 1024;
var $btn = $('.gHeadBtnArea');
if (w <= x) {
$btn.hide();
$(window).on( 'scroll', function () {
if ( $(this).scrollTop() < 60 ) {
$btn.slideUp('fast');
} else {
$btn.slideDown('fast');}
});
} else {
$btn.show();
$(window).on( 'scroll', function () {
$.noop;
});
}
});

Comments

[m.ts10806]

コードはマークダウンのcode機能を利用してください。またタイトル「jQueryの切り替えについて」だと質問内容とつながりがあまり感じられませんのでもう少し要件に寄せたタイトルに修正してください。

Answer 1

多数回呼ばれるイベント中でイベントを登録しない。
あと、投稿してすぐ退会とかしない。

var over = true;
var $btn = $('.gHeadBtnArea');
$(window).resize(function() {
    var w = window.innerWidth;
    var x = 1024;
    if (w <= x) {
        $btn.hide();
        over = false;
    } else {
        $btn.show();
        over = true;
    }
});
$(window).on('scroll', function() {
    if (over) {
        $.noop;
    } else {
        if ($(this).scrollTop() < 60) {
            $btn.slideUp('fast');
        } else {
            $btn.slideDown('fast');
        }
    }
});

Answer 2

最終的にどのような動作をさせたいかの記載がないので、間違っているかもしれません。

(function() {
  var x = 1024;
  var $btn = null;
  var flag = null;

  $(window)
    .on('load', function() {
      $btn = $('.gHeadBtnArea');
    })
    .on('resize', function() {
      var w = window.innerWidth;
      var _flag = w <= x;

      if (flag !== flag) {
        flag = _flag;
        $(window).off('scroll.button-toggle');

        if (flag) {
          $btn.hide();
          $(window)
            .on('scroll.button-toggle', function() {
              if ($(this).scrollTop() < 60) {
                $btn.slideUp('fast');
              } else {
                $btn.slideDown('fast');
              }
            })
            .trigger('scroll.button-toggle');
        } else {
          $btn.show();
        }
      }
    })
    .trigger('resize');
})();