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

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

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

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

jQuery

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

Q&A

解決済

2回答

2325閲覧

jQuery .remove() でDOMを消したい

hibikikudo

総合スコア238

JavaScript

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

jQuery

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

0グッド

1クリップ

投稿2018/02/01 05:32

前提・実現したいこと

クリックイベントでモーダルウィンドウとオーバーレイを出現・除去をトグルさせるプログラムにおいて、オーバーレイをDOMごと消したい。

発生している問題・エラーメッセージ

下記のコードを書きましたが、 .remove() の書き方が悪いのかオーバーレイのDOMが残ってしまい消えてくれません。
オーバーレイをフェイドアウト後にDOMごと消すにはどこを直せばいいでしょうか? どうぞ宜しくお願いいたします。
動くサンプル https://jsfiddle.net/vam4zwdd/

該当のソースコード

html

1<button type="submit" class="js-modal-open">保存</button> 2 3<div id="modal_content"> 4</div>

css

1#modal-overlay { 2 z-index: 1001; 3 display: none; 4 position: fixed; 5 top: 0; 6 left: 0; 7 width: 100%; 8 height: 100vh; 9 background-color: rgba(0, 0, 0, .5); 10} 11 12#modal_content{ 13 width: 100px; 14 height: 100px; 15 background:red; 16 position: absolute; 17 left: 50%; 18 top: 50%; 19 transform: translate(-50%, -50%); 20 padding: 15px 10px 30px; 21 display: none; 22 z-index: 1002; 23}

js

1"use strict"; 2 3var $modal_content = $("#modal_content"), 4 $modal_overlay = $("#modal-overlay"), 5 $modal_open = $(".js-modal-open"); 6 7//モーダル出現クリックイベント 8$modal_open.on("click", function(e) { 9 $(this).blur(); 10 //背景固定 11 $("body").css({ 12 position: "fixed", 13 width: "100%", 14 height: "100vh", 15 overflow: "hidden" 16 }); 17 //オーバーレイを出現させる 18 $("body").append( 19 '<div id="modal-overlay" tabindex="-1" aria-hidden="true"></div>' 20 ); 21 $("#modal-overlay").fadeIn(); 22 $modal_content.fadeIn(); 23 //閉じる 24 $(function() { 25 var close = $("#modal-overlay, .js-button--close"); 26 var modalset = $("#modal_content, #modal-overlay"); 27 close.on("click", function() { 28 //背景固定解除 29 $("body").attr({ 30 style: "" 31 }); 32 modalset.fadeOut(function() { 33 $modal_overlay.remove(); 34 close.off("click"); 35 }); 36 }); 37 }); 38}); 39

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

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

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

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

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

guest

回答2

0

ベストアンサー

先頭に$modal_overlay = $("#modal-overlay")がありますが、$による要素の検索は呼び出した時点で実行しますので、その時点で存在しなければ何も取れません。

オーバーレイをセットした後に、$modal_overlay = $("#modal-overlay")で保存しておきましょう。

投稿2018/02/01 05:44

maisumakun

総合スコア145183

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

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

0

//オーバーレイを出現させる
の前に、

javascript

1$modal_overlay = $("#modal-overlay"),

をしているのが原因だと思います。
closeするときに、要素を取得してくれば消すことができます。

javascript

1 modalset.fadeOut(function() { 2 $("#modal-overlay").remove();/* ここです */ 3 close.off("click"); 4 });

投稿2018/02/01 06:05

kszk311

総合スコア3404

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

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

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.48%

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

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

質問する

関連した質問