モーダルウィンドウをレスポンシブ対応させたい

Question

###前提・実現したいこと
leanModal.jsというプラグインを利用
モーダルウィンドウの表示位置を画面サイズに合わせて変えたい

###該当のソースコード
下記コード5行目の「top: 100,」をスマホ表示時に「top: 20,」にしたいです。

◆ jquery.leanModal.min.jsのソース

(function ($) {
  $.fn.extend({
    leanModal: function (options) {
      var defaults = {
        top: 100,
        overlay: 0.5,
        closeButton: null
      };
      var overlay = $("<div id='lean_overlay'></div>");
      $("body").append(overlay);
      options = $.extend(defaults, options);
      return this.each(function () {
        var o = options;
        $(this).click(function (e) {
          var modal_id = $(this).attr("href");
          $("#lean_overlay").click(function () {
            close_modal(modal_id)
          });
          $(o.closeButton).click(function () {
            close_modal(modal_id)
          });
          var modal_height = $(modal_id).outerHeight();
          var modal_width = $(modal_id).outerWidth();
          $("#lean_overlay").css({
            "display": "block",
            opacity: 0
          });
          $("#lean_overlay").fadeTo(200, o.overlay);
          $(modal_id).css({
            "display": "block",
            "position": "fixed",
            "opacity": 0,
            "z-index": 11000,
            "left": 50 + "%",
            "margin-left": -(modal_width / 2) + "px",
            "top": o.top + "px"
          });
          $(modal_id).fadeTo(200, 1);
          e.preventDefault()
        })
      });

      function close_modal(modal_id) {
        $("#lean_overlay").fadeOut(200);
        $(modal_id).css({
          "display": "none"
        })
      }
    }
  })
})(jQuery);

###試したこと
下記サイトを参考に自分なりに組み込んでみました。
https://idotdesign.net/blog/web/js/responsive-if-js/

leanModal: function (options) {
var windowWidth = $(window).width();
var windowSm = 480;
if (windowWidth > windowSm) {
      var defaults = {
        top: 100,
        overlay: 0.5,
        closeButton: null
      };
} else {
      var defaults = {
        top: 20,
        overlay: 0.5,
        closeButton: null
      };
}

###問題
エラーなどは発生していないが、効果が得られずスマホでもTop:100のままで表示されます。

Answer 1

動くサンプル：https://jsfiddle.net/w8rnunj1/

---

とりあえず動いているようです。
viewportを入れ忘れているとかありませんか？

【もう逃げない。HTMLのviewportをちゃんと理解する - Qiita】
http://qiita.com/ryounagaoka/items/045b2808a5ed43f96607

---

あと、ライブラリの引数側で処理できるので、ライブラリ自体のコードを書き換えるのはやめておいたほうが良いです。

var arg = { top : 200, overlay : 0.4 }; // ここを画面幅で切り替えれば良い
$("#trigger_id").leanModal( arg );

Comments

[tuckQ]

ご指摘ありがとうございます。

jsfiddleサンプルの正常動作も確認させていただきました。viewportは入れているので、原因がまだ分からないのですが、制作している実際のサイトが「http://jazztrigger.com/」になりまして、モーダル表示しているのが「レッスン内容」以下の「楽器」などのリンクになります。こちら見ていただけますでしょうか？

ライブラリ自体の書き換えは避けた方がよいのですね、引数渡しての処理に変更できるよう勉強してみます。

[kei344]

そもそもどこにも実行するスクリプトを書かれていないと思いますが、どこに書かれていますか？

[tuckQ]

私の理解が不足しているかもしれませんが、実行スクリプトはindex.htmlの50〜56行目に書かれているものと思っております。

[kei344]

これは失礼しました。
とりあえずChromeデベロッパーツールで問題なく動いているようです。動かないのはどの端末（及びOS）のどのブラウザ（及びバージョン）でしょうか。

【Web関係者は知っておくべきGoogleChromeデベロッパーツールの使い方 | Ocadweb】
http://ocadweb.com/web/googlechromedevelopertools

多分関係はないですが、rel="leanModal"のa要素の中に閉じていないdiv要素があります。
また、同じidが重複している箇所が多数あります。HTML中のidは重複できないため、classに変えるかidを重複しないように変える必要があります。

[tuckQ]

htmlに不備がたくさんあったようで失礼しました。divとid部分の修正をしたところ、iPhoneでうまく表示されるようになりました。ただ、Chromeのdevice toolbarでiPhone6などに設定してみてみるとtop100のままになっております。ちょっと気になりますが、一旦修正できたので、大変助かりました。いろいろアドバイスいただきありがとうございました。