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

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

ただいまの
回答率

90.03%

クリックした際に、先祖要素のdivに対してjavascriptでstyleを指定したい

受付中

回答 3

投稿

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

waco

score 4

前提・実現したいこと

スマホ用のメニューを作成しています。
メニューボタンをクリックした際に、先祖要素のdiv(id="gn-menu")に対してクラスを指定するか、style指定したいのですが、実装できません。

当方はhtmlはだいたい理解できるものの、javascriptに関しては、元々のものを改変する勉強を始めたばっかりのレベルです。
よろしくお願いいたします。

現在、メニュー動作は以下のサイトのものを引用
http://demo.mismithportfolio.com/test6/test6.html

上記のソースは画面上で常にヘッダーを上部に固定をする仕様になったいた。
ヘッダーが固定されるのが嫌だったため、その部分のCSSをコメントアウトした。

結果、メニューボタンをクリックしメニュー画面を開くと、メニュー画面は固定されるが
メニューバーの背景がスクロールされてしまっている。
なので、メニューボタンをクリックした場合にposition: fixed;をid="gn-menu"に対して行いたい。

該当のソースコード

<!-- スマホ用メニュー -->
<div id="gn-menu" class="gn-menu-main sp">
<a href="/" id="logo"><img src="./img/common/header_logo_sp.gif" alt="離婚弁護士 無料相談 法律事務所オーセンス"></a>
    <div class="gn-trigger">
        <a class="gn-icon gn-icon-menu">
            <span class="icon-bar"></span>
            <span class="icon-bar"></span>
            <span class="icon-bar"></span>
        </a>
    </div>

<!-- クリック後表示されるメニュー -->
    <nav class="gn-menu-wrapper">
        <div class="gn-scroller">

            <p class="field_link">アクセス</p>
            <ul class="nav-field gn-menu">
                <li><a href="./access/">六本木オフィス</a></li>
                <li><a href="./access/shinjuku.html">新宿オフィス</a></li>
                <li><a href="./access/kitasenju.html">北千住オフィス</a></li>
                <li><a href="./access/yokohama.html">横浜オフィス</a></li>
            </ul>

            <ul class="gn-menu gn-menu3">
                <li><a href="./contact/">お問い合わせ</a></li>
            </ul>

        </div>
    </nav>

</div>

試したこと

↓メニューボタンにあたる部分
<a class="gn-icon gn-icon-menu">

↓クリック時にCSSを付与したい箇所
<div id="gn-menu" class="gn-menu-main sp">

<script type='text/javascript'>  
$(function(){
$('#gn-menu').on('click' , "a.gn-icon-menu" , function(){
       $(this).closest("div#gn-menu").css("position","fixed");
   });
});
</script>
  • 気になる質問をクリップする

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 3

+4

普通にCSSを付与するだけならこれでいいと思います。親要素がIDである以上、クリックされたボタンから辿って親を探す必要はないのでこのようにしました。

  $('.gn-icon-menu').on('click',function(){
      $('#gn-menu').css('position','fixed');
  });

以下、おまけです

よく使いそうなものは変数に代入するといいです。今回程度のものでしたら気にしなくてもいいかもしれませんが、頭の片隅に入れていてほしいです。

違いは

[ 変数に代入する場合 ]
ページを開いた時、Javascriptは初めにHTML内の$('#gn-menu')を上から順に探してみつかった情報を変数内に保持する。それ以降は変数に代入された情報を元にアクセスするようになります。

[ 変数に代入しない場合 ]
Javascriptは必要なタイミングで毎回HTML内の$('#gn-menu')を上から順に探します。

$(function()
{
  //ここで変数に代入
  var gMenu = $('#gn-menu');
  $('.gn-icon-menu').on('click',function(){
      gMenu.css('position','fixed');
  });
});

今回、ボタンはクラスになっています。IDはHTMLの中でひとつというルールがあるのでJavascriptは上から順に探して見つかったら終了ですが、クラスはいくつあるか分からないのでJavascriptは上から下まですべて読みにいきます。なのでHTML上でひとつしか利用しなくても上から下まですべて探すという無駄な動作が増えてします。なので一つしか存在しないメニューでしたらIDを新たに割り振るというのも手だと思います。

また「Javascriptで使ってるよ」というのがわかりやすい名前だと尚いいと思います。

$(function()
{
  var gMenu = $('#gn-menu');
  $('#js-gn-icon-menu').on('click',function(){
      gMenu.css('position','fixed');
  });
});

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/08/21 18:04

    返信が遅くなり申し訳ありません。

    おまけ等も細かく記載いただきありがとうございます。
    ただアドバイスの通り記載してみましたが、スタイルは付与できませんでした。

    でも大変、勉強になりました!

    キャンセル

+2

現状のソースコードだけで質問者様の通り「Styleの指定」もしくは「Classの指定」
の動きになります。

// Styleの場合
  $(this).closest("div#gn-menu").css("position","fixed");
// Classの場合
  $(this).closest("div#gn-menu").addClass("clazzName");


ただ、javascriptで指定した以下のHTMLはCSSの指定が無ければ見た目上は存在
しない領域になりますが、その辺りのCSSはどうなっているのか次第かと。

<!-- -->
        <a class="gn-icon gn-icon-menu">
            <span class="icon-bar"></span>
            <span class="icon-bar"></span>
            <span class="icon-bar"></span>
        </a>

思うように見た目が変更されないのであれば、上記のHTML要素や他のHTML要素のCSS
設定の整合性がとれていないのではないでしょうか。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/08/21 17:45 編集

    返信が遅くなり申し訳ありません。
    お答えいただきありがとうございます!

    Classの場合の記載も書いていただきありがたかったです!!
    ただアドバイスいただいたものに書き換えて見ましたが、残念ながらCSSを指定できませんでした。
    kanimaruさんの言う通り、他との整合性が取れていないものと考えられます。

    他の要素を調べてみます!

    キャンセル

+2

「付与したい箇所」にはIDが振ってあるので(ページ内に1つであることが保証されている)下記で十分に思います。

$(function(){
$('#gn-menu').on('click' , "a.gn-icon-menu" , function(){
       $("#gn-menu").css("position","fixed");
   });
});

また、「親」や「子」を指定するのであれば、
parent()children()を利用した方がたどり易いですね。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/08/21 17:49

    返信が遅くなり申し訳ありません。
    お答えいただきありがとうございます!

    アドバイスいただいたjsに書き換えましたが残念ながら反応がありませんでした。

    アドバイスいただいた「親」や「子」をページをもとに調べてみます!

    キャンセル

  • 2017/08/22 08:59 編集

    同じIDが複数あるのでは・・?少なくとも私の方では回答内容で動作確認できています。

    キャンセル

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

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