jQuery窗口调整大小不起作用

Question

I'm trying to make a menu where sub menus are opened on hover but when on mobile (width < 800) sub menus should be triggered on click.

I made this jquery which is not working, it works on hover but when you re-size window its still triggered on hover not on click.

jquery:

$(document).ready(function () {

    $(".menu ul li").hover(function () {
        $('li > ul').not($(this).children("ul").slideToggle(200)).hide();
    });

});
$(window).resize(function () {
    if ($(window).width() <= 800) {
        $(".menu ul li").click(function () {
            $('li > ul').not($(this).children("ul").slideToggle(200)).hide();
        });
    }
}); 

CSS:

.menu {
  width: 100%;
  background: #333;
}
.menu:before,
.menu:after {
  content: "";
  display: table;
}
.menu:after {
  clear: both;
}
.menu > ul {
  width: 100%;
  list-style: none;
  padding: 0;
  margin: 0;
}
.menu > ul > li {
  float: left;
  background: #e9e9e9;
  padding: 0;
  margin: 0;
}
.menu > ul > li > a {
  text-decoration: none;
  padding: 1.5em 3em;
  display: block;
  outline: 0 none;
}
.menu > ul > li > ul {
  display: none;
  background: #333;
  padding: 20px 30px;
  position: absolute;
  width: 100%;
  z-index: 99;
  left: 0;
  color: #fff;
  margin: 0;
}
@media only screen and (max-width: 767px) {
  .menu ul > li {
    float: none;
    width: 100%;
  }
  .menu ul > li > ul {
    position: relative;
  }
}

HTML:

<div class="menu">
        <ul>
            <li><a href="#">Home</a>
                <ul>
                    This is also mega menu
                </ul>
            </li>
            <li><a href="#">Who are we?</a>
                <ul>
                    This is mega menu
                </ul>
            </li>
            <li><a href="#">Services</li></a>
                <li><a href="#">Contact</li></a>
        </ul>
    </div>

How can i do with jquery to change from on hover to on click when the window is resized or its below 800px?

Answer 1

use this script

$(document).ready(function () {

       $(".menu ul li").hover(function () {
          if ($(window).width() > 800) {

                $('li > ul').not($(this).children("ul").slideToggle(200)).hide();
          }

        });
         $(".menu ul li").click(function () {
          if ($(window).width() <= 800) {

                $('li > ul').not($(this).children("ul").slideToggle(200)).hide();
          }
        });

});

Answer 2

You can make use of .off() method to unbind the events attached and make sure to bind the other events with .on() :

$(document).ready(function() {
  $(".menu ul li").on('mouseenter', function() {
    $('> ul', this).stop().slideToggle(200); // this would do, as per context applied with "this"
  });

  $(window).resize(function() {
    if ($(window).width() <= 800) {
      $(".menu ul li").off().on('click', function() {
        $('> ul', this).stop().slideToggle(200); // this would do, context applied with "this"
      });
    }
  }).resize(); // trigger the resize() event on dom ready.
}); // <-----------wrap everything in the doc ready block

Plnkr demo

---

Even a better one:

// make a global function which toggles the submenus.
function submenutoggler() {
  $('> ul', this).stop().slideToggle(200); // .stop() will tackle the multiple clicks.
}

$(document).ready(function() {    
  $(window).resize(function() {
    if ($(window).width() <= 800) {
      $(".menu ul li").off().on('click', submenutoggler); // if mobile bind click
    }else{
      $(".menu ul li").on('mouseenter', submenutoggler); // if desktop bind mouseenter
    }
  }).resize(); // trigger the resize to execute at dom ready.

}); // <----wrap it within dom ready block.

Updated plnkr.

Answer 3

use like this

$(document).ready(function () {

    $(".menu ul li").on('hover',function () {
        $('li > ul').not($(this).children("ul").slideToggle(200)).hide();
    });

});
$(window).on('resize'function () {
    if ($(window).width() <= 800) {
        $(".menu ul li").on('click',function () {
            $('li > ul').not($(this).children("ul").slideToggle(200)).hide();
        });
    }
}); 

Answer 4

You need to unbind the event before the binding new event. I have modified your code see the fiddle here: http://jsfiddle.net/sachinkk/nu1dbp5a/

Also, made the changes in HTML. Hope this will helpful.

Make sure while loading the page, the size of a page should be more than 800px. (events are binding after page resizes)

 $(document).ready(function () { $(".menu ul li").hover(function () { $('li > ul').not($(this).children("ul").slideDown(200)).hide(); },function(){ $('li > ul').not($(this).children("ul").slideUp(200)).hide(); }); }); $(window).resize(function () { if ($(window).width() <= 800) { $(".menu ul li").unbind('hover'); $(".menu ul li").unbind('click'); $(".menu ul li").click(function (e) { $('li > ul').not($(this).children("ul").slideToggle(200)).hide(); }); } else{ if ($(window).width() > 800){ $(".menu ul li").unbind('click'); $(".menu ul li").unbind('hover'); $(".menu ul li").hover(function () { $('li > ul').not($(this).children("ul").slideDown(200)); },function(){ $('li > ul').not($(this).children("ul").slideUp(200)); }); } } }); 

 .menu { width: 100%; background: #333; } .menu:before, .menu:after { content: ""; display: table; } .menu:after { clear: both; } .menu > ul { width: 100%; list-style: none; padding: 0; margin: 0; } .menu > ul > li { float: left; background: #e9e9e9; padding: 0; margin: 0; } .menu > ul > li > a { text-decoration: none; padding: 1.5em 3em; display: block; outline: 0 none; } .menu > ul > li > ul { display: none; background: #333; padding: 20px 30px; position: absolute; width: 100%; z-index: 99; left: 0; color: #fff; margin: 0; } @media only screen and (max-width: 767px) { .menu ul > li { float: none; width: 100%; } .menu ul > li > ul { position: relative; } } 

 <script src="https://ajax.googleapis.com/ajax/libs/jquery/2.1.1/jquery.min.js"></script> <div class="menu"> <ul> <li><a href="#">Home</a> <ul> <li>This is also mega menu</li> </ul> </li> <li><a href="#">Who are we?</a> <ul> <li>This is mega menu</li> </ul> </li> <li><a href="#">Services</a></li> <li><a href="#">Contact</a></li> </ul> </div>