Correctly adding "use strict"; in this code

Question

I need to add "use strict"; in this code . when I try to add it stops working, this is a js of a responsive navigation menu

<script>
     $(function() {
        "use strict";
            var pull        = $('#pull');
                menu        = $('nav ul');
                menuHeight  = menu.height();

            $(pull).on('click', function(e) {
                e.preventDefault();
                menu.slideToggle();
            });

            $(window).resize(function(){
                var w = $(window).width();
                if(w > 320 && menu.is(':hidden')) {
                    menu.removeAttr('style');
                }
            });
        });

    </script>

What's the error message?

Bergi
– Bergi

2014-02-14 07:01:04 +00:00
Commented Feb 14, 2014 at 7:01 — Bergi
– Bergi, Commented Feb 14, 2014 at 7:01

elclanrs · Accepted Answer · 2014-02-14 07:00:23Z

4

You're creating global variables due to a syntax mistake. In strict mode these throw a reference error as the variable hasn't been declared:

var pull = $('#pull'), //<-- comma
    menu = $('nav ul'), //<-- comma
    menuHeight = menu.height();

answered Feb 14, 2014 at 7:00

elclanrs

94.2k21 gold badges137 silver badges171 bronze badges

Sign up to request clarification or add additional context in comments.

3 Comments

jasonscript Over a year ago

+1 If you view this in chrome's console window, it will actually give you the error: ReferenceError: menu is not defined

Bergi Over a year ago

@jasonscript: I like Opera's error message better: Assignment to unresolved reference in strict mode code: menu

jasonscript Over a year ago

@Bergi: that is better. My point was that if you're checking the console, you'll see the error and it's pretty straightforward to resolve this yourself

Collectives™ on Stack Overflow

Correctly adding "use strict"; in this code

1 Answer 1

3 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

3 Comments

Your Answer

Sign up or log in

Post as a guest

Related