要素をスクロールし、ウィンドウの上端から25ピクセルのときに固定します

Question

これは私のコードです。それはうまくいきますが、画像と重なることがあるバグがあります。これが他者のコード作成方法であるかどうかは疑問です。

var $window = $(window); 
 
var \t social = $('.social-links'); 
 
var \t elTop = social.offset().top - 25; 
 

 
$window.scroll(function() { 
 
    social.toggleClass('fixed', $window.scrollTop() > elTop); 
 
});

Answer 1

まずそれが選択された要素からjQueryオブジェクトを作成するために使われているとして、あなたは、ウィンドウの変数に$を使用しないでください。 try：

var window = $(window); 

私はそれを使用していないので、toggleClass関数で文を使用するかどうかはわかりません。

$window.scroll(function() { 
    if($(window).scrollTop() > elTop){ 
     social.toggleClass('fixed'); 
    } 
}); 

トグルクラスで状態を使用できる場合は、明らかにスペースを節約できます。

これを書いた後、スクロール時に情報を更新する必要があるので、ウィンドウ変数を設定する必要があるかどうか疑問に思っています。$（ウィンドウ）を使用して取得する必要があります。