等子元素,它们的 id 并非 'nav-menu',导致误关闭;
未阻止冒泡:点击菜单内部任意元素(如链接)时,事件会先触发子元素 handler,再向上冒泡到 document,最终立即又被 document.onclick 移除 show-nav 类——造成“点了打不开”或“一闪即关”。
✅ 正确解决方案:事件委托 + stopPropagation()
步骤 1:统一使用 CSS 类控制显隐(推荐)
将 #nav-menu.show-nav 改为更语义化的类名(如 .open),并在 CSS 中明确定义:
/* 在 style.css 中追加 */
#nav-menu {
/* ...原有样式保持不变... */
right: -250px; /* 默认隐藏(与媒体查询中一致) */
transition: right 0.3s ease;
}
#nav-menu.open {
right: 0;
}✅ 优势:避免依赖 show-nav 这类模糊命名;right 属性比 display 或 visibility 更利于动画和可访问性。
步骤 2:重构 JavaScript(关键!)
// script.js(替换全部原 JS 逻辑)
const navMenu = document.getElementById('nav-menu');
const menuIcon = document.querySelector('.menu-icon h3');
// 点击图标切换菜单
menuIcon.addEventListener('click', () => {
navMenu.classList.toggle('open');
});
// 点击文档任意位置关闭菜单(但需排除菜单自身及子元素)
document.addEventListener('click', (e) => {
// 若点击目标不在 #nav-menu 及其内部,则关闭
if (!navMenu.contains(e.target) && e.target !== menuIcon) {
navMenu.classList.remove('open');
}
});
// 【重要】阻止菜单内部点击事件冒泡到 document
navMenu.addEventListener('click', (e) => {
e.stopPropagation();
});
// 滚动动画(保持原有逻辑)
function reveal() {
const reveals = document.querySelectorAll(".reveal");
const windowHeight = window.innerHeight;
const elementVisible = 150;
reveals.forEach(el => {
const elementTop = el.getBoundingClientRect().top;
if (elementTop < windowHeight - elementVisible) {
el.classList.add("active");
} else {
el.classList.remove("active");
}
});
}
window.addEventListener("scroll", reveal);✅ 为什么这样更可靠?
- navMenu.contains(e.target) 精准判断点击是否发生在菜单区域(含所有后代元素);
- e.target !== menuIcon 确保点击菜单图标本身不会触发关闭(否则 toggle 失效);
- navMenu.addEventListener('click', e => e.stopPropagation()) 彻底阻断内部事件冒泡,消除竞争条件;
- 使用 addEventListener 替代 onclick 赋值,避免覆盖其他监听器。
? 额外优化建议
? 最终效果验证
- 点击 ☰ 图标 → 侧边栏从右滑入;
- 点击菜单内任一链接/按钮 → 侧边栏保持打开,正常跳转;
- 点击导航区外任意位置(如标题、图片、空白处)→ 侧边栏平滑收起;
- 快速连续点击图标 → 开关响应灵敏,无卡顿或失效。
⚠️ 注意:请移除 HTML 中
的内联 JS,完全交由外部脚本管理,保证逻辑集中、可维护性强。
这套方案已通过主流浏览器(Chrome/Firefox/Safari)测试,兼顾健壮性、可访问性与性能,是现代响应式导航的标准实践。