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

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

新規登録して質問してみよう
ただいま回答率
85.50%
HTML5

HTML5 (Hyper Text Markup Language、バージョン 5)は、マークアップ言語であるHTMLの第5版です。

JavaScript

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

jQuery

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

Sass

Sassは、プログラミング風のコードでCSSを生成できるスタイルシート言語です。 scss ファイルを、変換(コンパイル)してCSSファイルを作成します。

Q&A

解決済

4回答

5488閲覧

jQueryのクリックイベントが2回目以降、発火しない

T.Fujinaka

総合スコア7

HTML5

HTML5 (Hyper Text Markup Language、バージョン 5)は、マークアップ言語であるHTMLの第5版です。

JavaScript

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

jQuery

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

Sass

Sassは、プログラミング風のコードでCSSを生成できるスタイルシート言語です。 scss ファイルを、変換(コンパイル)してCSSファイルを作成します。

0グッド

1クリップ

投稿2018/11/27 16:58

編集2018/11/27 16:59

前提・実現したいこと

jQueryを用いて、タブのクリックによって表示するHTMLを切り替えることで動作する、表示切り替えタブの実装を行いたい。

前提(ソースコード参照)

toggleClassメソッドを用いてselected unselectedのクラスを入れ替えることで、今、どちらのタブがアクティブになっているかを識別できるようにし、
toggle()メソッドを用いて表示する中身のHTMLを切り替える方針にした。
その際、デフォルトHTMLでは一方にselectedクラスを、もう一方にunselectedクラスをあらかじめ付与しておいた。

jQueryでは、$('.unselected').on('click', function() {・・・}のようにクリックイベントを設置した。

問題

1回目のクリックイベントは正常に切り替えが動作したが、2回目以降のクリックイベントが1回目の要素上で発火してしまう。
おそらく、今のコードの書き方では、ページ読み込み時にunselectedクラスを持っている要素しかクリックイベントを持たない?
という理解でこのような問題が起きていると思っているのですが、この理解で正しいのでしょうか。
また、これを解決するためにはどうすればよいでしょうか。

該当のソースコード

html

1<div class="tab_container"> 2 <div class="change_tab left selected"> 3 <p>left</p> 4 </div> 5 <div class="change_tab right unselected"> 6 <p>right</p> 7 </div> 8</div> 9<div id="left_content"> 10 <p>left_content</p> 11</div> 12<div id="right_content"> 13 <p>right_content</p> 14</div>

SCSS

1.tab_container { 2 margin: 0 auto; 3 display: flex; 4 5 .change_tab { 6 flex-grow: 1; 7 flex-basis: 0; 8 border-bottom: 2px solid #d8dadf; 9 padding: 2px; 10 font-size: 20px; 11 } 12 13 .selected { 14 font-weight: bold; 15 color: black; 16 border-bottom: 2px solid #3385D9; 17 } 18 19 .unselected { 20 cursor: pointer; 21 } 22}

js

1$(document).on('turbolinks:load', function () { 2 $('.unselected').on('click', function() { 3 $('.left, .right').toggleClass("selected unselected"); 4 $('#left_content, #right_content').toggle(); 5 }); 6}); 7

試したこと

ここに問題に対して試したことを記載してください。

補足情報(FW/ツールのバージョンなど)

ここにより詳細な情報を記載してください。

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

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

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

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答4

0

ベストアンサー

質問文にあるとおり、動的に変更した DOM には、ページ表示時に設定したイベントが効かないため、質問文のような動作になっています。
他の回答にあるように、各要素が共有するクラス名でイベントを設定するか、jQuery のコードを以下のように修正することでこの問題は解決すると思います(動作確認用リンク)。

jQuery

1$(document).on("turbolinks:load", function() { 2 $(document).on("click", ".unselected", function() { 3 $(".left, .right").toggleClass("selected unselected"); 4 $("#left_content, #right_content").toggle(); 5 }); 6});

なぜ上記のコードでうまくいくのかは、以下の記事を読むとわかると思います。

投稿2018/11/27 17:15

編集2018/11/27 17:37
s8_chu

総合スコア14731

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

T.Fujinaka

2018/11/27 17:35

回答ありがとうございます。 `selected`クラスを持つ要素をクリックした時は何も発火して欲しくなかったので非常に助かりました! しかし上記のコードではイベントの発火自体が起こらず、以下のようにすれば正常に動きましたので念の為報告します。 ```jQuery $(document).on("click", ".unselected", function() { $(".left, .right").toggleClass("selected unselected"); $("#left_content, #right_content").toggle(); }); ```
guest

0

おそらく、今のコードの書き方では、ページ読み込み時にunselectedクラスを持っている要素しかクリックイベントを持たない?

この理解で正しいです。
unselectedクラス」ではなく、「タブが共通して持っているクラス」をフックにしてクリックイベントを設定すれば、選択状態に関わらず処理できます。

JavaScript

1$(document).on('turbolinks:load', function () { 2 $('.change_tab').on('click', function() { 3 $('.left, .right').toggleClass("selected unselected"); 4 $('#left_content, #right_content').toggle(); 5 }); 6});

投稿2018/11/27 17:12

NozomuIkuta

総合スコア1260

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

0

下記に変更してみてください

js

1$('body').on('click', '.unselected', function() { 2 // 処理 3});

投稿2018/11/27 17:23

akihiro3

総合スコア955

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

0

.change_tab にクリックイベントをつければよいのでは?

js

1 $('.change_tab').on('click', function() { 2 $('.left, .right').toggleClass("selected unselected"); 3 $('#left_content, #right_content').toggle(); 4 });

投稿2018/11/27 17:09

kei344

総合スコア69364

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問