ul/liを利用したアコーディオンメニュー: jQueryで「1つ開いたら他が全て閉じる」を実装

Question

お世話になります。

<ul>
	<li><a href="#!">リスト1</a></li>
	<li><a href="#!">リスト2</a>
		<ul>
			<li><a href="#!">サブリスト2-1</a></li>
			<li><a href="#!">サブリスト2-2</a></li>
			<li><a href="#!">サブリスト2-3</a></li>
		</ul>
	</li>
	<li><a href="#!">リスト3</a>
		<ul>
			<li><a href="#!">サブリスト3-1</a></li>
			<li><a href="#!">サブリスト3-2</a></li>
			<li><a href="#!">サブリスト3-3</a></li>
		</ul>
	</li>
</ul>

のようなリストがあり、普段は

ul li ul{
	display:none;
}

のようにサブリストを隠しています。

このリストを、以前教わった方法と同じように、

1. リストをクリックしたら、内包するサブリストを開く(slideToggle)
2. 開いているサブリストがあったら全て閉じる(hide)

という動作を期待して、

$(function(){
	$("ul li").on("click", function() {
			$(this).children().nextUntil().slideToggle("fast").end()
				.siblings("li").children().nextUntil().hide("fast");
	});
});

というコードを書いたところ、1) サブリストを開くところまではうまくいったのですが、2) 既に開いているサブリストを閉じることができませんでした。.siblings以下に問題があることは推測できるのですが、考えられる限りで書いたいずれの組み合わせでも、期待する動作は実現できませんでした。
(何も起こらないとか、全て消えるとか、自分の下のリストが全て消えるとか....)

どなたか、正しいコードを御教示いただけると嬉しいです。

Answer 1

こんにちは。

ご質問のコードで、.end() と .siblings("li") の間に
.parent()
を入れて以下のようにすると、

https://jsfiddle.net/jun68ykt/3cw2jv5u/

意図通りになりませんか？

参考になれば幸いです。

Comments

[Cystidia]

できました！ありがとうございました！！
1つ親まで戻らないとダメだったのですね...。

[jun68ykt]

解決されたようですね、よかったです！

[Cystidia]

勉強になりました！
複数要素を一気に選んだり、隣のものをいじったりする時は、いつも試行錯誤してます。

[Cystidia]

解決したはずだったのですが...、
<li><a>○○○○</a></li>の時はよいのですが、ここに
<li><a>○○○○<span class="arrow"></span></a></li>のようにspan(Font Awesomeで「＞」を拾ってきて.arrow:beforeで表示しています)を挟むと、サブメニューが「ない」項目のみspanが消えてしまいます。上の例だと、リスト2/3は「＞」が消えず、リスト1のみ、何かをクリックしたら「＞」が消えてしまいます。
liの子要素のうち、<ul>とその子要素のみ消す場合には、どうすればよいのでしょうか。

[Cystidia]

.filter("ul").hide("fast")であっさり解決いたしました。申し訳ありません。

[jun68ykt]

No problem です。
メソッドを何個もつなげたチェーンで、うまくいったら、なんかちょっと嬉しいですもんね。
何個もメソッドつなげて、うまくいくはずが、うまくいかないときは、いったん分解して、console.log に1個のメソッドごとに返された jqueryオブジェクトをログに出して確認するという地味な作業をやるしかないでしょう。ちなみに私も回答を書くときは、.end() が返したオブジェクトを console.log しました。