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

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

ただいまの
回答率

87.77%

アコーディオンの中のslick.jsが正常に初期化されない

解決済

回答 1

投稿

  • 評価
  • クリップ 1
  • VIEW 16K+

score 3725

前提・実現したいこと

・カルーセル表示にはslick.jsを使用している
・SPレイアウト(639px以下)ではアコーディオンパネルの中でカレンダーをカルーセル表示で見せる
・PCではアコーディオンなし/カルーセル表示なし
・ブラウザをリサイズしても上記表示を正常に切り替える

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

・ブラウザをリサイズした後アコーディオンを開くと、エラーが出てslick.jsの初期化が正しく行われない
エラー時のコンソール

該当のソースコード

<div class="js-accordion">
    <div class="js-acrTitle">アコーディオン開閉ボタン</div>
    <div class="item js-acrBody">
        <div class="jy-calendar" data-month="9">
            <div class="cal">
                <table>省略</table>
            </div><!-- /.cal -->
        </div><!-- /jy-calendar -->
    </div><!-- /.js-acrBody -->
</div><!-- /.js-accordion -->
//load
$(window).on('load', function(){
    accordionPanel01(); //アコーディオン開閉(※slick初期化含む)
});

//resize
var timer = false;
$(window).resize(function() {
    if (timer !== false) {
        clearTimeout(timer);
    }
    timer = setTimeout(function() {
        resetAccordionPane01(); //アコーディオン表示リセット
        initCalendar(); //カレンダー初期化(※slick初期化含む)
    }, 200);
});

/* カレンダー初期化処理
-------------------------------------------------- */
function initCalendar(){
    var $cal = $(".jy-calendar");
    var d = new Date(); //日時情報を取得
    var month = String(d.getMonth()+1); //当月を取得して文字列に変換
    var w = $(window).width();
    var size = 639;

    $cal.each(function(){
        var calMonthText = $(this).data("month"); //掲載月データを取得
        var calArray = String(calMonthText).split(","); //掲載月データを配列に変換
        var num = $.inArray(month, calArray); //当月が何番目にあるかを取得
        if(num < 0){
            num = 0; //当月が無かったら最初の月
        }
        if (w <= size) {
            $(this).slick({ //slick初期化
                initialSlide: num //当月をスライドの1番目に設定
            });            
        } else {
            $(this).unslick(); //slickを解除    
        }        
    });    
}

/* アコーディオン開閉処理
-------------------------------------------------- */
function accordionPanel01() {
    $(".js-acrTitle").each(function(){
        $(this).click(function(){

        var $eventTarget = $(this);

        if($(this).hasClass("op")){ //既にアコーディオンが開いている時の処理
            $(this).removeClass("op");
            $(this).next().stop().slideUp(400, function () {
                return;
            });
        }else{ //アコーディオンが閉じている時の処理
            $(this).addClass("op");
             if($eventTarget.next().has(".jy-calendar")){ //アコーディオンの中にカレンダーがあったら
                $eventTarget.next().find(".jy-calendar").slickSetOption(null, null, true); //slickの表示をリフレッシュ★ここでエラーが出る
                }
              });
        }

        });
    });
}

/* リサイズされた場合のアコーディオン表示処理
-------------------------------------------------- */
function resetAccordionPane01() {
    var w = $(window).width();
    var size = 639;

    $(".js-accordion").each(function(){
        if (w <= size) { //SPの時
            var display = $(this).find(".js-acrBody").css("display");
            if(display == "block"){//アコーディオンが開いていたら
                $(this).find(".js-acrTitle").addClass("op"); //開閉アイコンの状態を設定
            }
        } else { //PCの時
            //強制的にパネルを開く
            $(this).find(".js-acrBody").show(); 
            $(this).find(".js-acrTitle").removeClass("op"); 
        }        
    });
}

試したこと

最初はどういう状態だとうまく表示されないのか分からなかったので、エラーが再現されるタイミングを特定するためにコンソールの状態を確かめながらいろいろ試しました。  

・初期表示の状態で開閉 →問題なし
・アコーディオンを開いた状態でリサイズ後、開閉 →問題なし
・アコーディオンを閉じた状態でリサイズ後、開閉 →エラーでslickが正しく表示されない
・おかしくなった状態で再読み込みした後、開閉 →問題なし

というところまでは突き止めました。
リサイズ後、アコーディオンを開こうとすると、

Uncaught TypeError: Cannot read property 'removeClass' of null

というエラーが出て、slickのリフレッシュをしている箇所のコードがコケるようです。

if($eventTarget.next().has(".jy-calendar")){...}のif文を取ってみても変わりません。

カレンダー入りのアコーディオンはページ内に複数ありますが、1つにしてみても変化なしです。

ここからどうやってバグの原因を突き止めたら良いのか分からず、完全に行き詰まってしまいました。。。
どこかロジックがおかしいところや、エラーの原因になりそうなところが無いか、どなたかお助けいただけますと助かります。。。m(_ _)m

補足情報

MacのChromeとFirefoxでは確認していますがどちらも同じです。
その他不足情報等ありましたらご連絡ください。

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 1

checkベストアンサー

+1

initCalendar() で $(this).slick(); / $(this).unslick(); をしていますが、そのたびに同じ要素に .slick() を実行していることになるのでは。

load時に1回 .slick() して、それを変数に保存し、hoge.slick('setPosition'); とすればリロードできませんか?

【slick.jsを利用していて、初期状態をdisplay:none;にしたらレイアウト崩れる | SSSSSN】
http://sssssn.com/archives/288

関連するライブラリや全て提示してもらって、動作が再現できればもう少し回答の精度が上げれると思います。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2016/09/09 03:27

    ご回答と参考サイトの情報ありがとうございました。

    まさしくビンゴ!でした。
    ずっと「accordionPanel01()」と「resetAccordionPane01() 」ばかり見てあーでもないこーでもないとやっていて、initCalender()の方はノーマークだったので、目からウロコでした!!!

    slick(), unslick()の処理を「.slick-initialized」の有無で分岐するようにしたら正常に表示されるようになりました。

    重要なヒントをいただき、ありがとうございました。

    キャンセル

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

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

関連した質問

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