JS アコーディングメニュー 開閉時のアイコン切り替え

Question

前提

以下のJSでアコーディオンメニューを作成しています。また、現在地を取得してクラス名を付与しており、これによりページ読み込み時にクラス名is-mainNavKids--currentがついた場合には最初から子メニュー開いていたいためCSSで先述のクラス名にdisplay: block;を指定しています。さらに、CSSでアコーディオン閉じる時にはアイコンを＋に、開いた時にーに設定しています。

実現したいこと

初期状態でアコーディオンが開いていると、開いてる状態でアイコンが＋、閉じた時にーになってしまうので、初期状態で開いているアコーディオンにis-toggle-accNav--openを付与しーを表示させ、閉じた時に付与を取り消し+にしたい。

クラス名の付与以外をCSSだけで完結できるのであれば、そちらの方が好ましいです。（IE11対応必須）

該当のソースコード

//Nav
$(function(){
  $('.is-toggle-accNav').each(function(){
      $(this).on('click',function(){
          $(this).toggleClass('is-toggle-accNav--open');
          $("+.mainNavKids",this).slideToggle()
          return false;
      });
  });
});

//current
$(function(){
  $('.mainNav__item__link').each(function(){
    var $href = $(this).attr('href');
    if(location.href.match($href)) {
      $(this).parent('.mainNavKids__item').addClass('current');
      $(this).parents('.mainNav__item').addClass('current');
      $(this).parents('.mainNavKids').addClass('is-mainNavKids--current');
    } else {
      $(this).parent().removeClass('current');
    }
  });
});

.is-toggle-accNav:after {
  background: #33105f;
  content: '\f067';
  display: block;
  font-family: 'Font Awesome 5 Free';
  font-weight: 900;
  line-height: 60px;
  position: absolute;
  right: 0;
  text-align: center;
  top: 0;
  width: 60px;
}
.is-toggle-accNav.is-toggle-accNav--open:after {
  content: '\f068';
}
.is-mainNavKids--current {
  display: block;
}

<nav class="collapsibleNav">
  <button type="button" class="is-collapsibleNav-openBtn"></button>
  <ul class="mainNav">
    <li class="mainNav__item">
      <a href="#" class="mainNav__item__link is-toggle-accNav">親メニュー1（クリックで開く）</a>
      <ul class="mainNavKids is-mainNavKids--current">
        <li class="mainNavKids__item"><a href="./index.html" class="mainNav__item__link">サブメニュー1</a></li>
        <li class="mainNavKids__item"><a href="#" class="mainNav__item__link">サブメニュー1</a></li>
        <li class="mainNavKids__item"><a href="#" class="mainNav__item__link">サブメニュー1</a></li>
      </ul>
    </li>
〜略〜

問題のスクショ

以下、メニュー1にis-mainNavKids--currentが付与されている状態。メニュー2のアイコン表示が正しい状態。

Comments

[kyoya0819]

大幅に書き直しても大丈夫でしょうか？

[crigw]

現状の求めている動きを維持できてかつ複雑でなければ、大幅に書き直していただいても問題ありません。
また、こちらのJSとも喧嘩しなければ大丈夫です。
//collapsibleNav
$(function(){
document.querySelector('.is-collapsibleNav-openBtn').addEventListener('click', function(event) {
document.querySelector('.collapsibleNav').classList.toggle('is-collapsibleNav-closed');
}, false);
});

[miyabi_pudding]

jQueryのバージョンはなんでしょうか？
最新のバージョンだと、slideToggleが廃止されている可能性があります。
（私が検証した限りでは、3.4.1〜では、slideToggleメソッドがありませんでした。）

[crigw]

え、そうなんですか？
該当の3.4.1を読み込んでいるようです…動きはするのと廃止しているのとはまた別なのですね…

[miyabi_pudding]

なるほど・・・。
つまり、slideToggleの動作はしているということでしょうか？
こちらも再度検証します。

[kyoya0819]

3.4.1でも動きますね
https://codepen.io/asuchi0819/pen/xxbeKaO

[kyoya0819]

廃止もされていないみたいです。
https://api.jquery.com/slideToggle/

[miyabi_pudding]

asuchi0819さん、ありがとうございます。
こちらの検証不足ですね。
失礼しました。
再度検証していきます。

[miyabi_pudding]

crigwさん >
大変失礼しました。
slimバージョンを読み込んでいるのが原因でした。
どうやら、メソッドもかなり削減されてしまうようですね。
こちらも勉強になりました。

[crigw]

asuchi0819さん、miyabi_takatsukさん
SlideToggleについて、いろいろご確認や検証いただきありがとうございました。

Answer 1

あきらかな間違いは下記ですね。

(this).toggleClass('.is-toggle-accNav--open');

ドット(.)が余分です。

(this).toggleClass('is-toggle-accNav--open');

とりあえず上記を修正して希望の動作になるか確認してみてください。

追記

あと、初期状態で、カレントなら開いた状態、それ以外は閉じた状態にしておきたいのなら、CSSで設定しておきましょう。

.mainNavKids {
  display: none;
}
.is-mainNavKids--current {
  display: block;  
}

サンプルを作成したのでリンク置いておきます。

カレント開閉の処理で、カレントの時の処理で、親メニュー要素に is-toggle-accNav--openクラスを付与するコードが必要に思います。

DOMツリー構造をみて、辿っていけばいいでしょう。

子メニューがカレントの場合は、祖先の.mainNavKids の一つ前の要素にis-toggle-accNav--openクラスを付与すればいいので、下記のような感じで。

$(this).parents('.mainNavKids').prev().addClass('is-toggle-accNav--open');

親メニューがカレントの場合は、別途コードが必要になりますが、それはご自分で考えてみてください。

//Nav
$(function(){
  $('.is-toggle-accNav').each(function(){
      $(this).on('click',function(){
          $(this).toggleClass('is-toggle-accNav--open');
          $("+.mainNavKids",this).slideToggle()
          return false;
      });
  });
});

//current
$(function(){
  $('.mainNav__item__link').each(function(){
    var $href = $(this).attr('href');
    if($href === 'https://teratail.com') {
      $(this).parent('.mainNavKids__item').addClass('current');
      $(this).parents('.mainNav__item').addClass('current');
      $(this).parents('.mainNavKids').addClass('is-mainNavKids--current');
      $(this).parents('.mainNavKids').prev().addClass('is-toggle-accNav--open');
    } else {
      $(this).parent().removeClass('current');
    }
  });
});

動作確認用サンプル

Comments

[crigw]

ご指摘いただきました内容はすでに修正済みとなっております。
また今回の質問は`開閉時のアイコン切り替えについて`でして、カレント開閉の動作はすでに対応済みです…ですがご配慮頂きありがとうございました。

[hatena19]

カレント開閉の処理で、カレントの時の処理で、親メニュー要素に is-toggle-accNav--openクラスを付与するコードが必要に思います。

[crigw]

その記述方がわからずにいるためにこちらの質問を建てた次第です。

[hatena19]

DOMツリー構造をみて、辿っていけばいいでしょう。
子メニューがカレントの場合は、祖先の.mainNavKids の一つ前で、下記のような感じで。
$(this).parents('.mainNavKids').prev().addClass('is-toggle-accNav--open');
他にも辿り方はありますので一例です。

Answer 2

メニューの表示条件は@crigwさんの判定したいものに読み替えてください。
面倒なのでindexを判定していますので、無視してください。

親メニューのチェックと子メニューのチェックを分離すれば、'is-toggle-accNav--open'を付けるDOMツリー見えてくると思います。

$(function(){
  $('.is-toggle-accNav').each(function(){
      $(this).on('click',function(){
          $(this).toggleClass('is-toggle-accNav--open');
          $("+.mainNavKids",this).slideToggle()
          return false;
      });
  });
  // 親メニューの条件判定
  $('.mainNav__item__link'+'.is-toggle-accNav').each(function(index) {
    $("+.mainNavKids",this).hide(); // 一旦子メニューはすべて閉じる
    if (index == 2) {//<=ここの条件式はお好きなように
    	// 初期表示open
    	$("+.mainNavKids",this).show();
      $(this).addClass('is-toggle-accNav--open');
    }
  });
  // 親メニューがすべて閉じていた場合は、子メニューのチェックをする
  if ($('.mainNav__item__link'+'.is-toggle-accNav'+'.is-toggle-accNav--open').length < 1) {
    $('ul.mainNavKids .mainNav__item__link').each(function(index) {
      if (index == 0) {//<=ここの条件式はお好きなように
        // 初期表示open
        $(this).parents('li.mainNav__item').find('ul.mainNavKids').show();
        $(this).parents('li.mainNav__item').find('a.mainNav__item__link'+'is-toggle-accNav')
        	.addClass('is-toggle-accNav--open');
      }
    });
   }
});

Answer 3

結果的に以下1行の追加で希望していた動きになりました。
お二人ともありがとうございました。

//current
$(function(){
  $('a.mainNav__item__link, a.mainNavKids__item__link').each(function(){
    var $href = $(this).attr('href');
    if(location.href.match($href)) {
      $(this).addClass('current');
      $(this).parents('.mainNav__item').addClass('current');
      $(this).parents('.mainNavKids').addClass('current');
      $(this).parents('.mainNavKids').prev().addClass('is-toggle-accNav--open'); //この行の追加
    } else {
      $(this).parent().removeClass('current');
    }
  });
});