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

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

3 Comments

Related