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

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

ただいまの
回答率

89.97%

コンストラクタ関数で作ったライブラリのイベント発火のタイミングについて

受付中

回答 2

投稿

  • 評価
  • クリップ 1
  • VIEW 799

jkita1456

score 8

jsの勉強でコンストラクタ関数を使ってモーダルライブラリを作成しているのですが、
制作につまづいているのでアドバイスをいただけますでしょうか。

機能の詳細

  • 第一引数にtargetにイベントを発火させたい要素のID、第二引数のcontentにモーダルで表示したい要素のIDを指定
  • targetのクリックでイベントが発火。overlayとデリートボタン、contentのwrapper要素を生成
  • contentの要素は横幅と高さを取得し、画面中央に表示させる
  • deleteボタンとoverlayクリック時にモーダル非表示

困っていること

  • イベントを発火させる記述の方法
    (Modal.prototype.showContentとModal.prototype.hideContentの部分)

備考

  • contentにimg要素などを入れた場合に、画面中央表示させることはできました。
  • headタグ内に書かれているscriptは最終的には外部ファイルとして読み込み、html側は最小限の記述で済むようにしたいです。(インスタンスの生成のみ)
<!DOCTYPE html>
<html lang="ja">
<head>
<link rel="stylesheet" href="index.css">
<meta charset="UTF-8">
<title>サンプル</title>
<script src="https://ajax.googleapis.com/ajax/libs/jquery/2.1.4/jquery.min.js"></script>
<script>

function Modal(target, content) {
  this.target = $(target);
  this.content = $(content);
  this.width = this.getContentWidth();
  this.height = this.getContentHeight();
  this.content.css({
    zIndex: '999',
    position: 'absolute',
    top: '50%',
    left: '50%',
    display: 'none',
    marginTop: '-' +  this.height / 2 + 'px',
    marginLeft: '-' + this.width / 2 + 'px',
  }); 
}

Modal.prototype.getContentWidth = function() {
  var width = this.content.width();
  return width;
}

Modal.prototype.getContentHeight = function() {
  var height = this.content.height();
  return height;
}

Modal.prototype.showContent = function() {
  this.target.after('<div id="js-overlay" class="overlay"></div>');
  this.target.after('<div id="js-button" class="button-delete"></div>');
  this.content.wrap('<div class="content-wrapper"></div>');
  this.content.css({
    display: 'block',
  });
}

Modal.prototype.hideContent = function() {
  this.target.after('');
  this.target.after('');
  this.content.wrap('');
  this.content.css({
    display: 'none',
  });
}

</script>
</head>
<body>

<div class="button-container">

  <div id="button-left" class="button">
    <p>サンプルボタンA</p>
  </div>
  <div id="content-left">
  </div>

  <div id="button-center" class="button">
    <p>サンプルボタンB</p>  </div>
  <div id="content-center">
  </div>

  <div id="button-right" class="button">
    <p>サンプルボタンC</p>
  </div>
  <div id="content-right">
  </div>

</div>

<script>
  var modalLeft = new Modal('#button-left', '#content-left');
  var modalCenter = new Modal('#button-center', '#content-center');
  var modalRight = new Modal('#button-right', '#content-right');
</script>
</body>
</html>
  • 気になる質問をクリップする

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

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

  • 退会済みユーザー

    2016/12/06 10:09

    こちらの質問が他のユーザから「やってほしいことだけを記載した丸投げの質問」という指摘を受けました
    「質問を編集する」ボタンから編集を行い、調査したこと・試したことを記入していただくと、回答が得られやすくなります。

回答 2

+2

とりあえず this.target.after(''); の部分は間違っています。「targetの次の要素に '' を追加する」という動作になり、this.target.after('<div id="js-overlay" class="overlay"></div>'); を消すという動作にはなりません。

【.after() | jQuery 1.9 日本語リファレンス | js STUDIO】
http://js.studio-kingdom.com/jquery/manipulation/after

【.after() | jQuery API Documentation】
http://api.jquery.com/after/

【jQueryで要素を挿入できるメソッドがたくさんあるので整理してみた | bl6.jp】
http://bl6.jp/web/javascript/organize-method-insert/

クリックイベントは $('p').on( 'click', function() {} ); $('p').click( function() {} ); のどちらかで設定出来ます。

【【jQuery】イベント発火方法 - Qiita】
http://qiita.com/kazu56/items/95f78b53baeb1170daeb

【jQuery 便利なonを使おう(on click) - Qiita】
http://qiita.com/shizuma/items/d561f37f864c3ebb5096

モーダルライブラリのよくある実装は、モーダル用のHTMLを1つだけ作って、それの中身を書き換える方法が多い気がします。質問文のコードで書かれているようにそれぞれHTMLを用意する実装も考えられますが、その場合は <div id="js-overlay" class="overlay"></div> のように id属性を使わないほうが良いです。同一ページ内に同じid属性が複数あるとHTMLの文法違反になるほか、JavaScriptで取得する際の妨げになる可能性があります。( idをキーに取得する場合に1つ目のみしか取得できない 例⇒ https://jsfiddle.net/qh0v3xh4/ )

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

0

イマイチなにをしたいのかわかりませんが、こういうことでしょうか?
とりあえず発火はしますが挙動についてはなんともいえません

$(function(){
  var modals={};
  modals["button-left"]= new Modal('#button-left', '#content-left');
  modals["button-center"]= new Modal('#button-center', '#content-center');
  modals["button-right"]= new Modal('#button-right', '#content-right');
  $('#button-left,#button-center,#button-right').on('click',function(){
    modals[$(this).attr('id')].showContent();
  });
});

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

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

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