2016-04-09 16 views
0
$(window).bind('scroll', function() { 
    if ($(this).scrollTop() > 520) { 
     $('.element').fadeIn(); 
    } if ($(this).scrollTop() < 520) { 
     $('.element').stop().fadeOut(); 
    } 
}); 

私はこれをまとめて、520の後に要素がフェードインし、520未満の場合フェードアウトするようにしました。私の質問は次のとおりです。別のものにする?fadeIn、fadeOut jQueryコードスニペット

+1

は、私はあなたが何を意味するかわからない「であり括弧」正しいが、あなたは、構文を意味するならば、を大丈夫だよ。 2番目の 'if'文を' else'に変更することをお勧めします。それ以外の場合は、 'scrollTop' = 520の場合がまったくありません。 –

+1

あなたが '}のようなelseを見逃しましたelse if($(this).scrollTop()<520){' –

答えて

1

私は、これは、このセクションのソートの話題から外れていると思うが、SOのこの部分をチェックアウト:Code Review

構文はよさそうだが、他の人が言ってきたように、あなたがあれば、他にそれを変更したい場合があります、または他のあなたが欲しいものに応じて。ここで

は、/ else文場合は私が見つけた最高のリソースです:

MDN If. . . Else

関連する問題