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

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

ただいまの
回答率

90.50%

  • JavaScript

    16457questions

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

  • HTML

    8996questions

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

  • jQuery

    6719questions

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

  • CSS

    5805questions

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

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

解決済

回答 3

投稿

  • 評価
  • クリップ 0
  • VIEW 953

lista

score 7

前提・実現したいこと

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

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

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

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

該当のソースコード

<div id="contents">
  <div id="box1">Box1</div>
  <ul id="list">
    <li>AAA</li>
    <li>BBB</li>
    <li>CCC</li>
    <li>DDD</li>
    <li>EEE</li>
  </ul>
  <div id="box2">Box2</div>
</div>
* {
  margin: 0;
  padding: 0;
}

#box1 {
  height: 1000px;
  background: #ccc;
}

#box2 {
  height: 500px;
  background: #ccc;
}

#list {
  list-style: none;
  overflow: hidden;
}

#list li {
  width: 20%;
  height: 100px;
  border: 1px solid #333;
  float: left;
  box-sizing: border-box;
}

.fadeUpOrder {
  position: relative;
  bottom: -100px;
  opacity: 0;
  transition: all .5s;
}

.fadeUpOrder.active {
  bottom: 0;
  opacity: 1;
}
$(function(){
  $('#list li').addClass('fadeUpOrder');
  var elem = $('.fadeUpOrder');

  $(window).on('scroll', function(){
    var scr = $(window).scrollTop();
    var winHeight = $(window).height();
    elem.each(function(i){
      if(scr + winHeight > $(this).offset().top + 100){
        $(this).delay(i * 100).queue(function(){
          $(this).addClass('active').dequeue();
        });
      }else{
        $(this).removeClass('active');
      }
    });
  });
});


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

試したこと

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

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

jQuery 3.2.1

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

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

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

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 3

checkベストアンサー

+6

改善してみました。

  • スクロールの判定をli要素ではなく#list要素で行う
  • queueによる待機ではなく、transition-delayを使う
  • 各要素それぞれにdelayが設定されているので、一斉にaddClassでactiveを追加しても大丈夫
  • スクロールをガクガク素早くやっても必ず引っ込んだ状態からスタートするように、ひっこむ際は一瞬で消えるようにdelayとdurationを0にする
$(function() {
  $('#list li').addClass('fadeUpOrder');
  $(window).on('scroll', function() {
    var scr = $(window).scrollTop();
    var winHeight = $(window).height();
    if (scr + winHeight > $('#list').offset().top + 100) {
      $('#list li').each(function(i) {
        $(this).css('transition-delay', (i * 100) + 'ms');
        $(this).css('transition-duration', '.5s');
      }).addClass('active');
    } else {
      $('#list li').each(function(i) {
        $(this).css('transition-delay', '0ms');
        $(this).css('transition-duration', '0s');
      }).removeClass('active');
    }
  });
});


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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/10/19 17:06

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

    キャンセル

+5

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

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

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

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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/10/19 17:06

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

    キャンセル

+2

変数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 17:38

    終わってたのかー

    キャンセル

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

  • ただいまの回答率 90.50%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る

  • JavaScript

    16457questions

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

  • HTML

    8996questions

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

  • jQuery

    6719questions

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

  • CSS

    5805questions

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