モーダル の背景を固定化したい

Question

モーダル を実装しております。
モーダル 自体はきちんと動きます。
そこで、モーダル を開くボタンをおし、モーダル が出てきた際、bodyの部分はmodal_bgとしてグレーのレイヤーがかかるようにしています。
そこで、問題なのが、このモーダルが開いた際、bodyとグレーのレイヤーがスライドしてしまいます。
モーダル が開いた際、背景は固定されているようにしたいです。

自分で行ったこととしては、
そこで、調べつつ
height: 100vh;
overflow-y: hidden;
を追加したり、自分でJSを書いたりしましたが、うまくいきませんでした。
私としては、モーダルオープンを押した時に、cssにすでに記載しているmodal-openのクラスが付与され、背景が固定化し、モーダル閉じるを押すと、クラスが除去されるというように実装したつもりでした。
おそらくJSの書き方に問題があるのかもしれません。
何か悪い点など見当たればご教授頂きたいです。よろしくお願いいたします。
（文字数の都合上、モーダル のデザインのCSSは省略させて頂いております。）

  <div class="work-photo">
                <div class="container">
                    <h3 class="work-title">Photography</h3>
                <div class="work-items">
                    <div class="work-item">
                        <p class="work-item-title"></p>
                        <a class="js-modal-open" href="" data-target="modal04"><img src="/img/work-1-icon.jpg" alt=""></a> 
                    </div>
                    <div class="work-item">
                        <p class="work-item-title"></p>
                        <a class="js-modal-open" href="" data-target="modal05"><img src="/img/work-2-icon.JPG" alt=""></a>
                    </div>
                    <div class="work-item">
                        <p class="work-item-title"></p>
                        <a class="js-modal-open" href="" data-target="modal06"><img src="/img/work-3-icon.jpg" alt=""></a>
                    </div>
                </div>
            </div>

                <div id="modal04" class="modal js-modal">
                    <div class="modal__bg js-modal-close"></div>
                    <div class="modal__content">
                        <img src="/img/work-1.jpg" alt="photo-1">
                        <a href="" class="js-modal-close">close</a>
                    </div>
                </div>
                <div id="modal05" class="modal js-modal">
                    <div class="modal__bg js-modal-close"></div>
                    <div class="modal__content">
                        <img src="/img/work-2.jpg" alt="photo-2">
                        </div>
                        <a href="" class="js-modal-close">close</a>
                    </div>
                </div>
                <div id="modal06" class="modal js-modal">
                    <div class="modal__bg js-modal-close"></div>
                    <div class="modal__content">
                        <img src="/img/work-3.jpg" alt="photo-3">
                        </div>
                        <a href="" class="js-modal-close">close</a>
                    </div>
                </div>

            </div>
           </div>

jQuery(function(){
  jQuery('.js-modal-open').each(function(){
      jQuery(this).on('click',function(){
          var target = jQuery(this).data('target');
          var modal = document.getElementById(target);
          jQuery(modal).fadeIn();
          return false;
      });
  });
  jQuery('.js-modal-close').on('click',function(){
      jQuery('.js-modal').fadeOut();
      return false;
  }); 
});



jQuery(function($){
  $('.js-modal-open').on('click', function () {
    $body.toggleClass('modal-open');
    $('.js-modal-close').on('click', function () {
      $body.removeClass('modal-open');
    });
  });
});

.modal{
    display: none;
    height: 100vh;
    position: fixed;
    top: 0;
    width: 100%;
    z-index: 999;

.modal__bg{
    background: rgba(0,0,0,0.8);
    position: absolute;
    width: 100%;
    height: 100%;
}
.modal__content{
    background: #fff;
    left: 50%;
    padding: 50px 35px;
    position: absolute;
    top: 50%;
    transform: translate(-50%,-50%);
    width: 70%;
    max-width: 500px;
    height: 70%;
    overflow-y: scroll;
    z-index: 999;
    img {
        margin-bottom: 30px;
    }
    .work-item-text {
        text-align: center;
        width: 85%;
        margin: 0 auto;
        .fas {
            font-size: 25px;
            padding-right: 10px;
        }
        .work-place {
            padding: 10px 0;
        }
        .work-camera {
            padding-bottom: 30px;
        }
        .work-explain {
            padding-bottom: 20px;
            line-height: 25px;
        }
    }
    .work-url {
        padding-bottom: 30px;
    }


    .js-modal-close {
        display: block;
        text-align: center;
        color: $text-color;
        padding: 10px;
        border: 1px $text-color solid;
        width: 50%;
        margin: 0 auto;
        &:hover {
            background-color: $text-color;
            color: #fff;
        }
    }
}
}

    

body .modal-open {
    height: 100vh;
    overflow-y: hidden;
}

Comments

[kuma_kuma_]

jQueryを使用されているようなので
指定で使っているCSSファイルとJSファイル名を教えてもらえませんか？

[MIYABITSUJIMOTO]

コメントありがとうございます。
/css/style.css
/js/script.js
という感じでよろしいでしょうか？

[kuma_kuma_]

できればjQueryのファイルを教えてほしかったんだけど...

[hatena19]

CSSはSCSSですか。もしそうなら、タグにSCSSも追加してください。
あと、変数の宣言部分も追加してください。

Answer 1

修正依頼にコメントしましたが、SCSSを使っているならタグにSCSSを追加してください。
また、SCSSコードの変数宣言の部分を追加しておいてください。

間違いがCSSとJSに一つずつありますのでそれを修正すれば想定の動作になるでしょう。

$body を $('body') に修正

jQuery(function($){
  $('.js-modal-open').on('click', function () {
    $('body').toggleClass('modal-open');
    $('.js-modal-close').on('click', function () {
      $('body').removeClass('modal-open');
    });
  });
});

body .modal-open を body.modal-open に修正

/* body .modal-open { */
body.modal-open {
    height: 100vh;
    overflow-y: hidden;
}

.modal-openはbodyに設定されているので半角空白は削除してください。
半角空白があるとbodyの子要素という意味になります。

Comments

[MIYABITSUJIMOTO]

ご回答ありがとうございます
お返事大変遅くなり申し訳ございません。
通知に気づかずいました。
先ほど試してみると、トップページから全くスクロールできなくなりました。逆にモーダル の時はきちんと背景がスクロールされるようになってしまいました。。。。。
逆ということですかね

[hatena19]

当方で作成したサンプルでは問題なく動作してます。
https://codepen.io/hatena19/pen/JjKdrWw