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

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

ただいまの
回答率

89.51%

data属性を利用したモーダルメニューを作りたい。

解決済

回答 2

投稿

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

dream

score 31

現在は、これまで作成してきたクラスを用いたメニューとは異なる
data属性を用いたメニューを作成しているのですが、何も変化が
見られなくて困っています。申し訳ないのですがどなたか知恵を
お貸しいただければ幸いです。

コード <body>
  <div class="nav-menu" data-open="false">
    <nav class="inner">
      <ul class="nav-menu-list">
      <li class="list-item">
        <a href="about.html" target class="barba-link"></a>
      </li>
    </ul>
    </nav>
  </div>
  <header class="header">
    <div class="header-menu">
      <button class="header-menu-btn">
        <span class="menu-text">MENU</span>
      </button>
    </div>
  </header>


  <script src="https://ajax.googleapis.com/ajax/libs/jquery/3.3.1/jquery.min.js"></script>
  <script src="js/script.js"></script>
</body>
コード  .header-menu-btn {
    position: relative;
    width: 47px;
    height: 42px;
    display: block;
    &:before,&:after {
        content: "";
        width: 85%;
        height: 4px;
        background: #000;
        position: absolute;
        left: 50%;
        transform: translate(-50%,0);
    }
    &:before {
        top: 5px;
    }
    &:after {
        top: 20px;
    }
}

.menu-text {
    font-family: fantasy;
    letter-spacing: 1;
    display: block;
    line-height: 1;
    position: absolute;
    top: 30px;
    left: 10%;
}

.nav-menu[data-open="false"] {
    position: absolute;
    top: 0;
    left: 0;
    width: 100%;
    height: 100%;
    overflow: hidden;
    pointer-events: none;
    opacity: 0;
    transition: opacity .2s ease-out;
}

.nav-menu[data-open="true"] {
    display: block;
    opacity: 1;
    background: #000;
    position: absolute;
    left: 0;
    top: 0;
    width: 100%;
    height: 100%;
}

.list-item {
    list-style-type: none;
}

.nav-menu-list {
    max-height: none;
    width: 100%;
}

.inner {
    width: 100%;
    display: block;
}
コード  $(function(){
    $(".header-menu").click(function(){
        $("body").removeAttr("data-open","false"),
        $("body").attr("data-open","true")
    return false;
    }); 
    });

目標としては、アイコンがクリックされると非表示が解除され、
trueの内容が表示されるといったイメージです。

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

質問への追記・修正、ベストアンサー選択の依頼

  • m.ts10806

    2020/03/24 13:51

    初期値をtrue/false切り替えた場合、想定の動作はしているのでしょうか?
    …ただ、不確かで申し訳ないですがdata属性は値ですら非同期的な動作をしてたと記憶にあり、CSSの切り替えのようなことに使うのはあまり適してないように思います。
    (不確かなので後程検証はしてみます)

    キャンセル

  • m.ts10806

    2020/03/24 14:08

    上記、勘違いでした。簡易コードで試しただけですが、attrで変化したstyleは効きますね。
    ブラウザ開発ツールの検証で属性が想定通り変化しているか確認してみてください。

    キャンセル

  • dream

    2020/03/24 20:21

    わかりました、やってみます!

    キャンセル

  • dream

    2020/03/25 08:55

    無事イメージ通りの動きになってくれました。
    多忙なところお付き合いいただきありがとうございました。

    キャンセル

回答 2

checkベストアンサー

0

JavaScript側を、以下のように書き換えたら動きそうです。
(true / falseなので、「!」を使うなどもっとスマートな方法はあるかもしれませんが)

$(function() {
  $(".header-menu").click(function() {
    if ($(".nav-menu").attr("data-open") == "true") {
      $(".nav-menu").attr("data-open", "false");
    }
    else {
      $(".nav-menu").attr("data-open", "true");
    }

    return false;
  }); 
});

CSSを見ると、.nav-menu[data-open="false"]など.nav-menuに対しての記述はあるものの、
JavaScript側では$("body").removeAttr("data-open","false"),などbodyに対して操作をしようとしているようでした。
そのためJavaScript側で、$(".nav-menu")に対して記述を追加するようにしました。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2020/03/25 11:38

    迅速な対応恐れ入ります。おかげさまでうまくいきました、
    ありがとうございます。

    キャンセル

  • 2020/03/26 18:50

    回答していただいたものを一通り試した結果、他の方のもうまくいきました
    故、甲乙付け難いのですが一番先に答えていただきましたので、感謝の意を
    込めてベストアンサーを付けさせていただきます。

    キャンセル

  • 2020/03/26 20:35

    ありがとうございます!

    キャンセル

0

雑ですがこんな感じですかね?

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2020/03/24 20:21

    <style>
    #modal{display:none}
    #modal[data-open="true"]{display:block;}
    </style>
    <script>
    window.addEventListener('DOMContentLoaded', ()=>{
    document.querySelector('#hoge').addEventListener('click',e=>{
    var modal=document.querySelector('#modal');
    var open=modal.dataset["open"];
    if(open=="true"){
    modal.dataset["open"]="false";
    e.target.textContent="on";
    }else{
    modal.dataset["open"]="true";
    e.target.textContent="off";
    }
    });
    });
    </script>
    <div id="hoge">on</div>
    <div id="modal">modal</div>

    キャンセル

  • 2020/03/25 09:42

    ご多忙のところの回答恐れ入ります。一つ気になった点を申し上げますと、
    'click'の後のe=>はどのようなことを意味しているのでしょうか?

    キャンセル

  • 2020/03/25 09:56

    eはイベントを渡すための引数で、click処理情報を引き継ぎます。
    e.targetを利用するのが目的でこの場合のe.targetは
    クリックされた#hoge自体をさします

    キャンセル

  • 2020/03/25 11:34

    なるほど、そういった意味なんですね。
    詳しい説明ありがとうございます。とても為になりました。

    キャンセル

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

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