モーダルウィンドウの挙動について

Question

javascript初心者です。
現在自作モーダルを作成しており、モーダルウィンドウのアニメーションで躓きましたので投稿いたしました。
モーダルは自作のため参考サイトはなく、動きの部分で参考にしたのが下記になります。
https://sole-color-blog.com/blog/707/
上記のサイトでボタンをクリックした時の処理で、animationスタイルを付与して、開く時、閉じる時のアニメーションを別々にしておりましたので、上記を参考にanimationを付与しましたが、開く時はアニメーションがかかり、閉じる時はアニメーションがかかりませんでした。
閉じる時のスタイル付与は問題なく行われており、原因がわかりません、、、

原因がわかる方やその他改善案がある方はお知恵をお貸し下さい。
宜しくお願い致します。

<section>
    <img alt="" src="/corporate/bsl/topic/csr_03.png">
    <div class="p-csr-txtBox">
        <p class="tag">CRS</p>
        <h3>サンプルテキストサンプルテキストサンプルテキストサンプルテキスト</h3>
        <p class="p-csr-txt">
            サンプルテキストサンプルテキストサンプルテキストサンプルテキストサンプルテキストサンプルテキストサンプルテキストサンプルテキストサンプルテキストサンプルテキストサンプルテキストサンプルテキスト
        </p>
        <button class="p-modalBtn" value="date01">続きを読む</button>
    </div>
</section>



<article class="p-modalDateBox on" id="date01">
    <div class="p-modal-bg"></div>
    <div class="p-whiteSpace">
        <div class="heading">
            <p class="tag">aaaa</p>
            <p class="date">2020.06.24</p>
        </div>
        <button class="close">閉じる</button>
    </div>
</article>

.p-modalDateBox {
    position: absolute;
    opacity: 0;
    z-index: -1;
    transition: 1s;
}

.p-modalDateBox.on {
    opacity: 1;
    display: block;
    position: absolute;
    width: 100%;
    height: 100%;
    top: 0;
    left: 0;
    z-index: 1;
    transition: 1s;
}

/*モーダルウィンドウ*/
.p-modal-bg {
    background: #00000063;
    position: fixed;
    width: 100%;
    height: 100%;
    top: 0;
    left: 0;
    z-index: -1;
}

.on .p-modal-bg {
    z-index: 2;
}

.p-whiteSpace {
    position: absolute;
    width: 90%;
    height: 90%;
    background-color: #FFF;
    left: 0;
    right: 0;
    top: 80px;
    bottom: 0;
    margin: auto;
    overflow-y: scroll;
    z-index: 3;
    opacity: 0;
    transform: translate(0, -500px);
}

.on .p-whiteSpace {
    opacity: 1;
    top: 0;
    transform: translate(0, 0px);
}

@keyframes modal {
    0% {
        top: -100%
    }

    100% {
        top: 50%
    }
}

@keyframes modalClose {
    0% {
        top: 50%
    }

    100% {
        top: -100%
    }
}

//外部ファイル読み込み
axios.get("modal.txt").then(response => {
    const body = document.querySelector('.modal');
    body.innerHTML = response.data
})


//モーダルウィンドウ

const close = document.querySelectorAll(".close");
const modalBox = document.querySelectorAll(".p-modalDateBox");
const modalBg = document.querySelectorAll(".p-modal-bg");
const whiteSpace = document.querySelectorAll(".p-whiteSpace");

console.log(modalBox);
modalBox.forEach(element => {
    console.log(element)
    element.classList.remove("on");

    close.forEach(c => {
        c.addEventListener("click", function () {
            element.classList.remove("on");
        });
    });

    modalBg.forEach(m => {
        m.addEventListener("click", function () {
            element.classList.remove("on");
            whiteSpace.forEach(w =>{
            console.log("animate");
                w.style.animation = "modalClose 2s";
            })
        });
    });
});


document.body.addEventListener('click', function (event) {
    if (event.target.className == "p-modalBtn") {
        const v = event.target.value;
        const modelId = document.querySelector("#" + v);
        console.log(modelId);
        modelId.classList.add("on");
        whiteSpace.forEach(w =>{
            w.style.animation = "modal 2s";
        })
    }
}, false)

Answer 1

閉じる時はアニメーションがかかりませんでした

いえ、確認したところ、閉じる時にもアニメーションがかかっています。
ただ、

element.classList.remove("on");

↑としているため、

.on .p-whiteSpace {
    opacity: 1;
}

↑のスタイルが効かなくなり、

.p-whiteSpace {
    opacity: 0;
}

↑が効いて、透明になっているため、見えないのと、

.p-modalDateBox {
/*
    width：auto;
    height: auto;
    （初期値） 
*/
}

.p-modalDateBox.on {
    width: 100%;
    height: 100%;
}

↑となっているため、.onが外れた時に、.p-modalDateBoxがトランジションがかからないままサイズを失ってしまうのが原因です。

---

解決方法ですが、モーダルのスタイルがかなり混乱していますので、整理した方がいいと思います。
正直、コードを見ただけでは「正しい動作」がなんなのか、想像がつきません

• アニメーションにするのかトランジションにするのかを決める
• どの要素にアニメーション｜トランジションをかけるのか決める
• アニメーション｜トランジションを適用するプロパティを決める

Comments

[sarisari]

動かすことが出来ました！！
確かにLhankor_Mhyさんのご指摘通り、アニメーション、トランジションが混在しており自分でも把握出来ておりませんでした。。
その点もどちらを利用するのか決めて見やすく改善いたします。
ありがとうございました！