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

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

ただいまの
回答率

90.35%

  • JavaScript

    17500questions

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

  • jQuery

    7103questions

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

アコーディオンメニューの開閉を示すアイコンがうまく機能しない

解決済

回答 1

投稿 編集

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

kitikiti

score 5

メニューをクリックするとアイコンがマイナスになりません。

下記のようなコードなのですが、どこを改善すればよいでしょうか?
どこがネックになっているかわからなかったのでjavascript得意な方見ていただけませんでしょうか?

プラス(+)マイナス(-)機能が機能しません。

閉じている時
開いているとき

<script type="text/javascript" src="https://ajax.googleapis.com/ajax/libs/jquery/1.9.1/jquery.min.js"></script>
<script type="text/javascript">
$(function(){
  $("#menu > li > p.accordion_icon").removeClass('active');
  $("#menu > li > ul").hide();
  $("#menu > li").on("click",function(){
    $(this).siblings("li").find('> ul').slideUp();
    $(this).siblings("li").find('> p.accordion_icon').removeClass('active');
    $(this).find('> ul').slideToggle(800);
    $(this).find('> p.accordion_icon').toggleClass('active');
  });
});
</script>
<ul id="menu">
<li><a href="#">●×●×</a></li>
<li><a href="#" class="menus">●×●×<p class="accordion_icon"><span></span><span></span></p></a>
<ul class="child" >
<li><a href="#">●×●×</a></li>
<li><a href="#">●×●×</a></li>
<li><a href="#">●×●×</a></li>
</ul>
</li>
</ul>
ul#menu li{

    background-color: #f8f8f8;
    border-top:  solid 1px #e3e3e3;
    border-bottom: solid 1px #e3e3e3;
}

ul#menu li a{
    color: #000000;
    text-decoration: none;
    display: block;
    padding: 15px; 

}

ul#menu li a:hover{
    text-decoration: underline;
    background: #e3e3e3;
}

ul#menu li a:before {
    display: block;
    content: "";
    position: absolute;
    -webkit-transform: rotate(45deg);
    transform: rotate(45deg);
    left: 235px;
    width: 8px;
    height: 8px;
    margin-top: 4px;
    background: #393939;
}
ul#menu li a:after {
    display: block;
    content: "";
    position: absolute;
    -webkit-transform: rotate(45deg);
    transform: rotate(45deg);
    left: 233px;
    width: 8px;
    height: 8px;
    margin-top: -12px;
    background: #f8f8f8;
}

ul#menu li a:hover:after {
    background: #e3e3e3;
}

ul#menu li a.menus:before {
    display: block;
    content: "";
    position: absolute;
    -webkit-transform:  rotate(0deg);
    transform: rotate(0deg);
    left: 231px;
    width: 0px;
    height: 0px;
    margin-top: 0px;
    background: #393939;
    font-weight: bold;
}

ul#menu li a.menus:after {
    display: block;
    content: "";
    position: absolute;
    -webkit-transform: rotate(0deg);
    transform: rotate(0deg);
    left: 233px;
    width: 0px;
    height: 0px;
    margin-top: -12px;
    background: #f8f8f8;
}

ul#menu li a.menus:hover:after {
    background: none
}

/*アイコンプラスマイナス*/
.accordion_icon,
.accordion_icon span {
    display: inline-block;
    transition: all .4s;
    box-sizing: border-box;
}
.accordion_icon {
    position: relative;
    width: 25px;
    height: 25px;
    float: right;
    margin-right: -7px;
    /* margin-top: -90px; */
}
.accordion_icon span {
    position: absolute;
    left: 6px;
    width: 50%;
    height: 2px;
    background-color: black;
    border-radius: 4px;
    -webkit-border-radius: 4px;
    -ms-border-radius: 4px;
    -moz-border-radius: 4px;
    -o-border-radius: 4px;
}
.accordion_icon span:nth-of-type(1) {
    top: 5px;
    transform: rotate(0deg);
    -webkit-transform: rotate(0deg);
    -moz-transform: rotate(0deg);
    -ms-transform: rotate(0deg);
    -o-transform: rotate(0deg);
}
.accordion_icon span:nth-of-type(2) {
    top: 5px;
    transform: rotate(90deg);
    -webkit-transform: rotate(90deg);
    -moz-transform: rotate(90deg);
    -ms-transform: rotate(90deg);
    -o-transform: rotate(90deg);
}
/*+、-切り替え*/
.accordion_icon.active span:nth-of-type(1) {
    display:none;
}
.accordion_icon.active span:nth-of-type(2) {
    top: 5px;
    transform: rotate(180deg);
    -webkit-transform: rotate(180deg);
    -moz-transform: rotate(180deg);
    -ms-transform: rotate(180deg);
    -o-transform: rotate(180deg);
}




ul#menu ul.child{
    display: none;
    padding: 0;
    background: #e3e3e3;
    width: 275px;
    -moz-box-sizing: border-box;
    -webkit-box-sizing: border-box;
    -o-box-sizing: border-box;
    -ms-box-sizing: border-box;
    box-sizing: border-box;

}

ul#menu ul.child li{
    border-top:  none;
    border-bottom: none;
    margin-left: 70px;
    padding-left: -15px !important;
    background: #e3e3e3;
    color: #000000;
    list-style-type: disc;
}


ul#menu ul.child li a{
    background: #e3e3e3;
    display: block;
    text-decoration: none;
    color: #000000;
    -moz-box-sizing: border-box;
    -webkit-box-sizing: border-box;
    -o-box-sizing: border-box;
    -ms-box-sizing: border-box;
    box-sizing: border-box;
}

ul#menu ul.child li a:hover{
    text-decoration: underline;
}


ul#menu ul.child li a:before {
    display: none;
}
ul#menu ul.child li a:after {
    display: none;
}
ul#menu ul.child li a:hover:after {
    display: none
}
コード
  • 気になる質問をクリップする

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 1

checkベストアンサー

+1

提示HTML中に accordion_icon が出てくるのは一か所だけですが、li の子要素ではないため引っかかりません(activeになりません)。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/07/04 12:24

    今回は1か所のみになります。
    1か所のみだと書き方が変わるのでしょうか。

    キャンセル

  • 2018/07/04 13:04

    数は関係ないですが?

    キャンセル

  • 2018/07/04 13:17

    よくわからず申し訳ありません。
    どのようにすればいいのでしょうか。

    キャンセル

  • 2018/07/04 13:23

    該当のp.accordion_icon を li の子要素にすればいいのではないでしょうか。

    ところで、提示されたコードは自分で確認してみたのでしょうか?
    デザインからはこれがメニューであることがわかりません。

    キャンセル

  • 2018/07/04 13:43

    もちろん、HTMLを変えるのではなくスクリプトを、"> p.accordion_icon" から "> a > p.accordion_icon" にすることも考えられます。

    キャンセル

  • 2018/07/04 13:48

    イメージをアップしました。

    キャンセル

  • 2018/07/04 13:49

    ありがとうございます。
    スクリプトの変更でなりました。
    ご協力に感謝します。

    キャンセル

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

  • JavaScript

    17500questions

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

  • jQuery

    7103questions

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