原生JavaScript实现的模态框点击按钮后不显示问题求助

问题原因 #

1. CSS规则优先级与顺序冲突

你当前的CSS代码中，.open-modal 类的规则写在了 .modal-overlay 类规则的前面，两个选择器都是单类选择器，权重完全相同，CSS解析时后面的规则会覆盖前面的规则。所以即使你给元素加上了open-modal类，.modal-overlay里定义的visibility: hidden和z-index: -10依然会覆盖前者的配置，导致模态框看不到，和你手动取消这两个属性就正常显示的现象完全吻合。

2. JS执行时机早于DOM加载完成

如果你的JS代码是放在<head>标签里，或者放在HTML按钮、模态框元素的上方，代码执行时DOM还没渲染完成，querySelector就会找不到对应的元素，事件绑定失效，就会出现点击打开、关闭按钮都无响应的情况。

解决方案 #

• 调整CSS规则顺序：把.open-modal的规则挪到.modal-overlay规则的后面即可，修改后CSS参考：

.modal-overlay {
  position: fixed;
  top: 0;
  left: 0;
  height: 100%;
  width: 100%;
  display: flex;
  place-items: center;
  justify-content: center;
  background: rgba(76, 52, 90, 0.15);
  z-index: -10;
  visibility: hidden;
}

/* open-modal规则移到.modal-overlay后面，同权重下后定义的规则生效 */
.open-modal {
  visibility: visible;
  z-index: 100;
}

.modal-container {
  position: relative;
  text-align: center;
  background: #fff;
  font-weight: 300;
  border-radius: 10px;
  padding: 10px;
  height: 400px;
  display: flex;
  align-items: center;
}

你也可以直接提高.open-modal的优先级，不受规则顺序影响，写法如下：

/* 两个类组合的选择器权重高于单个类，不会被.modal-overlay的规则覆盖 */
.modal-overlay.open-modal {
  visibility: visible;
  z-index: 100;
}

• 调整JS加载时机：把你的JS代码整个挪到HTML的</body>标签之前，或者用DOM加载完成事件包裹JS代码，避免元素还没渲染就执行查询逻辑，示例如下：

document.addEventListener('DOMContentLoaded', function() {
  const modalOverlay = document.querySelector(".modal-overlay")
  const modalClose = document.querySelector(".close-btn")
  const modalBtn = document.querySelector('.btn-primary')

  modalBtn.addEventListener("click", function() {
    modalOverlay.classList.add("open-modal");
  });

  modalClose.addEventListener("click", function() {
    modalOverlay.classList.remove("open-modal")
  });
})

修改完成后即可正常实现模态框的打开、关闭功能。

内容的提问来源于stack exchange，提问作者Gad Ashell