jQuery undefined variable in IE8

Question

I've created a jQuery mini script for slider. It's work perfectly in Firefox, Chrome, IE9 but when I hover in IE7, IE8. It's show an alert error (Undefined current_hover_item).

$('.nav-item').hover(function(){
        var current_hover_item = $(this).children().attr('rel');
        $('.show-item').hide();
        $(current_hover_item).fadeIn(1000);
        $(this).animate({"width": "+=10px", opacity: 0.6}, 500, function(){
             $(this).animate({"width": "-=10px", opacity: 1}, 500);
        });
}, function(){
        $(current_hover_item).fadeOut(1000);
        return false;
});

I tried remove (var) keywords but when I'm hover, it won't show everything. Please help me. Thanks.

Community · Accepted Answer · 2017-05-23 12:04:43Z

2

current_hover_item is undefined in the second function, since the definition in the first function is in a closure.

You need to declare current_hover_item in both functions. Or, since you only use the variable once in each function, remove it entirely, like this:

$('.nav-item').hover(function(){
        $('.show-item').hide();
        $($(this).children().attr('rel')).fadeIn(1000);
        $(this).animate({"width": "+=10px", opacity: 0.6}, 500, function(){
             $(this).animate({"width": "-=10px", opacity: 1}, 500);
        });
}, function(){
        $($(this).children().attr('rel')).fadeOut(1000);
        return false;
});

Or preferably refactor your code.

edited May 23, 2017 at 12:04

CommunityBot

11 silver badge

answered Jul 4, 2012 at 19:58

Chris Laplante

29.7k18 gold badges110 silver badges137 bronze badges

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

1 Comment

Nguyen Trong Tan Over a year ago

Thanks. I've tried your code but in my context, it's doesn't work.

Losbear · Accepted Answer · 2012-07-04 19:59:16Z

0

I was having this problem earlier today. I fixed it by declaring the variable outside of any jquery functions (including .ready()) and then set it where you need it (where you are currently declaring it).

answered Jul 4, 2012 at 19:59

Losbear

3,3231 gold badge34 silver badges29 bronze badges

Collectives™ on Stack Overflow

jQuery undefined variable in IE8

2 Answers 2

1 Comment

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

1 Comment

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related