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

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

ただいまの
回答率

90.47%

  • JavaScript

    17018questions

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

  • jQuery

    6929questions

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

アコーディオンを全部閉じた状態にしたい

解決済

回答 4

投稿

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

eggplant_bc

score 11

jQueryでアコーディオンを作ったのですが、全部を閉じた状態に出来なくて困っています。

デフォルトでは全部閉じた状態で、どれかひとつをクリックで開き、その後別のひとつをクリックして開くとさっきまで開いていたのを閉じる、という挙動にしました。
最後開いている場所をもう一度クリックすると一度閉じるのですがまた開いてしまうという現象が起きています。

$(function() {
      $(".inside").hide();   // .insideはアコーディオンの隠れている部分
      $(".ac_head").click(function() {   // .ac_headはクリックする部分
            $(".inside").slideUp(500);
            $(".inside", this).slideDown(500);
      });
});

if文で「すでに開いている箇所をクリックしたら閉じる」というように書くのかなと思ったのですが、どのように書けば良いのか分かりません。
もしご存知の方いらっしゃいましたら、ご教示お願いいたします。

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 4

checkベストアンサー

+5

こんな感じですかね?

$(function() {
  $(".inside").hide();
  $(".ac_head").on('click',function() {
    $(this).find(".inside").slideToggle(500)
    $(this).siblings().find(".inside").slideUp(500);
  });
});
<ul>
<li class="ac_head">1
<ul class="inside">
<li>1-1</li>
<li>1-2</li>
<li>1-3</li>
</ul>
</li>
<li class="ac_head">2
<ul class="inside">
<li>2-1</li>
<li>2-2</li>
<li>2-3</li>
</ul>
</li>
<li class="ac_head">3
<ul class="inside">
<li>3-1</li>
<li>3-2</li>
<li>3-3</li>
</ul>
</li>
</ul>

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/04/20 10:33 編集

    siblingsの存在忘れてました・・・。
    これなら一発信シンプルでいけますよね。

    キャンセル

  • 2018/04/20 11:05

    兄弟要素選択なんてあったのかー

    キャンセル

  • 2018/04/20 13:13

    yambejpさん、ご回答いただきありがとうございました!
    まだまだ勉強し始めたばかりなもので、.siblingsや.findを初めて知りました…。
    yambejpさんから教えていただいたコードで解決しました。
    とても勉強になりました。ありがとうございます!

    キャンセル

+4

if(!$("セレクタ").is(":hidden")){
などどうでしょう

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/04/20 13:28

    Dotさん、ご回答ありがとうございます!
    :hiddenという指定の仕方でも出来るんですね!
    知らなかったので、とても勉強になりました。
    ありがとうございます!

    キャンセル

+4

そりゃ、.insideのセレクタで二回同時に処理をしているんだからそうなりますよ。
それと、HTML構成も記載したほうがいいと思います。
(jQueryでのセレクタの構成をどう回答すればいいかわかりずらいので)

現状から、推測して、
Dotさんの回答を使い、下記でいかがでしょうか?

$(function() {
  $(".inside").hide();   // .insideはアコーディオンの隠れている部分
  $(".ac_head").click(function() {   // .ac_headはクリックする部分
    // クリックした対象のアコーディオン要素を格納しておく
    var target = $(this).parent().find(".inside");
    $(".inside").each(function(){
      if(!$(this).is(":hidden")){
        $(this).not(target).stop().slideUp(500);
      }
    });

    if (target.is(":hidden")) {
      target.stop().slideDown(500);
    } else {
      target.stop().slideUp(500);
    }

  });
});

一度、クリックした要素の.insideと、それ以外の.insideと処理を分けています。
こうすることによって、挙動を分けてわかりやすくなります。
また、stop()を実行することによって、
何度もクリックされた際の、変な挙動を抑えることができます。
(これを入れないと、連続クリックした回数分アニメーション処理が実行されてしまう)

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/04/20 13:16

    miyabi_takatsukさん、ご回答いただきありがとうございます!
    HTMLを記載しなかった件はすみませんでした…。
    コードも詳しく書いていただき、とても勉強になります。
    ありがとうございました!

    キャンセル

+1

slideToggle()を使えば簡単に実現可能かと思われますが...

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/04/20 13:26

    mercy5さん、ありがとうございます!
    slideToggle単体で使っていたのですが、そうすると複数の開閉が可能なアコーディオンになってしまったのでslideUpとslideDownを使っていました。
    ご回答ありがとうございました!

    キャンセル

関連した質問

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

  • JavaScript

    17018questions

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

  • jQuery

    6929questions

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