覆盖点击关闭菜单需要双击而不是单击

时间:2016-04-17 08:26:25

标签: javascript jquery

我有一个幻灯片菜单,我想在用户点击叠加层时关闭它。菜单关闭但是要再次打开它我必须两次点击切换而不是一次,我在哪里错了?谢谢。展示此slideoutMenu

的示例页面



function expandOverlay() {
  var overlay = document.createElement("div");
  overlay.setAttribute("id", "overlay04");
  overlay.setAttribute("class", "overlay04");
  document.body.appendChild(overlay);
}

function restore() {
  document.body.removeChild(document.getElementById("overlay04"));
}

// create menu variables
var slideoutMenu = $('.slideout-menu');
var slideoutMenuWidth = $('.slideout-menu').width();

$(document).ready(function() {
  $('.slideout-menu-toggle').on('click', function(event) {
    event.preventDefault();
    // toggle open class
    slideoutMenu.toggleClass("open");

    // slide menu
    if (slideoutMenu.hasClass("open")) {
      slideoutMenu.animate({
        left: "0px"
      });
      expandOverlay();
    } else {
      slideoutMenu.animate({
        left: -slideoutMenuWidth
      }, 250);
      restore();
    }
  });

});

window.addEventListener('mouseup', function(event) {
  var box = document.getElementById('menu_s');
  if (event.target != box && event.target.parentNode != box) {
    slideoutMenu.animate({
      left: -slideoutMenuWidth
    }, 250);
    restore();
  }
});

<nav>
<ul>
  <li><a href="#" class="lag slideout-menu-toggle">open menu</a></li>
</ul>
</nav>

<!--begin slideout menu-->
<div id="menu_s" class="slideout-menu">
<h3>Last Week<a href="#" class="slideout-menu-toggle">&times;</a></h3>
<div class="fslide"></div>
<ul>
  <li><a href="http://dundaah.com/">Dundaah</a></li>
  <li><a href="http://pics.dundaah.com/">Pics</a></li>
  <li><a href="http://blog.dundaah.com/">Blog</a></li>
  <li><a href="http://music.dundaah.com/">Music</a></li>
</ul>
</div>
&#13;
&#13;
&#13;

1 个答案:

答案 0 :(得分:5)

我认为这里的问题是在点击叠加后,类打开仍然在toggleMenu上。

因此,如果您在关闭后单击打开菜单,则click事件将从toggleMenu中删除open类,从而导致if语句的else部分被执行(即关闭菜单)。

这就是为什么在菜单打开之前需要点击2次,如果单击叠加层将其关闭。

点击叠加层关闭菜单时,您可能会尝试toggleClass(“open”)。

相关问题