質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.50%
JavaScript

JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

jQuery

jQueryは、JavaScriptライブラリのひとつです。 簡単な記述で、JavaScriptコードを実行できるように設計されています。 2006年1月に、ジョン・レシグが発表しました。 jQueryは独特の記述法を用いており、機能のほとんどは「$関数」や「jQueryオブジェクト」のメソッドとして定義されています。

HTML

HTMLとは、ウェブ上の文書を記述・作成するためのマークアップ言語のことです。文章の中に記述することで、文書の論理構造などを設定することができます。ハイパーリンクを設定できるハイパーテキストであり、画像・リスト・表などのデータファイルをリンクする情報に結びつけて情報を整理します。現在あるネットワーク上のほとんどのウェブページはHTMLで作成されています。

CSS

CSSはXMLやHTMLで表現した色・レイアウト・フォントなどの要素を指示する仕様の1つです。

Q&A

解決済

3回答

963閲覧

jQuery 順番にフェードインさせる処理が2回目以降、順番通りにならない

lista

総合スコア13

JavaScript

JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

jQuery

jQueryは、JavaScriptライブラリのひとつです。 簡単な記述で、JavaScriptコードを実行できるように設計されています。 2006年1月に、ジョン・レシグが発表しました。 jQueryは独特の記述法を用いており、機能のほとんどは「$関数」や「jQueryオブジェクト」のメソッドとして定義されています。

HTML

HTMLとは、ウェブ上の文書を記述・作成するためのマークアップ言語のことです。文章の中に記述することで、文書の論理構造などを設定することができます。ハイパーリンクを設定できるハイパーテキストであり、画像・リスト・表などのデータファイルをリンクする情報に結びつけて情報を整理します。現在あるネットワーク上のほとんどのウェブページはHTMLで作成されています。

CSS

CSSはXMLやHTMLで表現した色・レイアウト・フォントなどの要素を指示する仕様の1つです。

0グッド

0クリップ

投稿2017/10/19 07:23

###前提・実現したいこと
jQueryを用いて「スクロールして画面内に対象の要素が入ったら順番にフェードインさせる」処理を書きました。

2回目以降も、綺麗に左から順番にフェードインするようにしたいです。

###発生している問題・エラーメッセージ
スクロールをして最初に対象の要素が画面内に入ったとき、期待通りに左から順番にフェードインします。
しかし、ある程度上までスクロールして戻り、再度下にスクロールしていくと今度は順番通りにフェードインせず、バラバラです。
1個目の要素がフェードインしなかったり、4個目より5個目の方が早くフェードインしたり。

###該当のソースコード

HTML

1<div id="contents"> 2 <div id="box1">Box1</div> 3 <ul id="list"> 4 <li>AAA</li> 5 <li>BBB</li> 6 <li>CCC</li> 7 <li>DDD</li> 8 <li>EEE</li> 9 </ul> 10 <div id="box2">Box2</div> 11</div>

CSS

1* { 2 margin: 0; 3 padding: 0; 4} 5 6#box1 { 7 height: 1000px; 8 background: #ccc; 9} 10 11#box2 { 12 height: 500px; 13 background: #ccc; 14} 15 16#list { 17 list-style: none; 18 overflow: hidden; 19} 20 21#list li { 22 width: 20%; 23 height: 100px; 24 border: 1px solid #333; 25 float: left; 26 box-sizing: border-box; 27} 28 29.fadeUpOrder { 30 position: relative; 31 bottom: -100px; 32 opacity: 0; 33 transition: all .5s; 34} 35 36.fadeUpOrder.active { 37 bottom: 0; 38 opacity: 1; 39}

JavaScript

1$(function(){ 2 $('#list li').addClass('fadeUpOrder'); 3 var elem = $('.fadeUpOrder'); 4 5 $(window).on('scroll', function(){ 6 var scr = $(window).scrollTop(); 7 var winHeight = $(window).height(); 8 elem.each(function(i){ 9 if(scr + winHeight > $(this).offset().top + 100){ 10 $(this).delay(i * 100).queue(function(){ 11 $(this).addClass('active').dequeue(); 12 }); 13 }else{ 14 $(this).removeClass('active'); 15 } 16 }); 17 }); 18});

状況が再現できるコード
https://jsfiddle.net/5Lnnpnhq/1/

###試したこと
開発者ツールで対象の要素のクラスの付与状態がどうなっているか監視しました。
スクロールして戻った際、画面外にも関わらずactiveクラスがついている場合があります。
.queue()を使っているせいだと思うのですが、どのように対処したらいいかがわかりません。

###補足情報(言語/FW/ツール等のバージョンなど)
jQuery 3.2.1

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答3

0

ベストアンサー

改善してみました。

  • スクロールの判定をli要素ではなく#list要素で行う
  • queueによる待機ではなく、transition-delayを使う
  • 各要素それぞれにdelayが設定されているので、一斉にaddClassでactiveを追加しても大丈夫
  • スクロールをガクガク素早くやっても必ず引っ込んだ状態からスタートするように、ひっこむ際は一瞬で消えるようにdelayとdurationを0にする

javascript

1$(function() { 2 $('#list li').addClass('fadeUpOrder'); 3 $(window).on('scroll', function() { 4 var scr = $(window).scrollTop(); 5 var winHeight = $(window).height(); 6 if (scr + winHeight > $('#list').offset().top + 100) { 7 $('#list li').each(function(i) { 8 $(this).css('transition-delay', (i * 100) + 'ms'); 9 $(this).css('transition-duration', '.5s'); 10 }).addClass('active'); 11 } else { 12 $('#list li').each(function(i) { 13 $(this).css('transition-delay', '0ms'); 14 $(this).css('transition-duration', '0s'); 15 }).removeClass('active'); 16 } 17 }); 18});

https://jsfiddle.net/5Lnnpnhq/3/

投稿2017/10/19 07:52

masaya_ohashi

総合スコア9206

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

lista

2017/10/19 08:06

頂いたコードで期待通りの動作をするようになりました! 解説も、書いて頂いたコードもとても分かりやすくて助かりました。ありがとうございました!
guest

0

スクロール時のタイミングの問題で、

  1. activeをセットするタイマーが入る
  2. 上にスクロールする
  3. イベントでactiveが外れる
  4. 1でのタイマーが発動して、activeが付く

というような動きになってしまっています。3で.clearQueue()を入れてみれば、取りあえず動くようになります(修正したJSFiddle)。

なお、scrollイベントは頻繁に起こるので、そのたびごとにDOM操作をするのは少し負荷が大きいです。タイマーなどで動作するタイミングを絞ったほうがいいかもしれません。

投稿2017/10/19 07:55

maisumakun

総合スコア145121

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

lista

2017/10/19 08:06

なるほど、そのような原因でうまく動作していなかったのですね。 scrollイベントでの発火も対策を考えてみます。ありがとうございました!
guest

0

変数scrをログってみました。

900 _display:90:7 897 _display:90:7 887 _display:90:7 871 _display:90:7 863 _display:90:7 854 _display:90:7 836 _display:90:7 823 _display:90:7 808 _display:90:7 791 _display:90:7 784 _display:90:7 771 _display:90:7 757 _display:90:7 753 _display:90:7 751 _display:90:7

かなり細かくイベントが発生しているので、removeClassメソッドでクラスを除去したに、過去のイベントで発生していたキューのコールバックが呼ばれaddClassメソッドが実行されている、ということはありそうです。
そうなった場合、$(this).offset().topの値が100ずれるわけですから、次にスライドインしてくるタイミングもその分ずれるでしょう。

もし、そうだとすると、delayメソッドの待ち時間の少ない手前の要素は比較的影響を受けにくいと予想されますので、あるところまでスクロールした時に「1個目の要素がフェードインしなかったり、4個目より5個目の方が早くフェードインしたり」することが起き得るかと思います。

投稿2017/10/19 08:37

Lhankor_Mhy

総合スコア35865

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問