Question

The replace() global is not working in IE7, but works within IE8. Its a bunch of products which I am toggling on and off.

If I remove the .replace() it works in IE7.

Script

$('li.prod').toggle(true);

$('li.prod:visible').each(function(i){

    i && ((i+1)%4 || $(this).addClass('prod-end'));

});

$('li.button').click(function (){

    $('li.button').removeClass('active');

    $(this).addClass('active');

$('li.prod').removeClass('prod-end');

if ( $('li#all').hasClass('active')) {

    $('li.prod').toggle(true);

    $('li.prod:visible').each(function(i){

            i && ((i+1)%4 || $(this).addClass('prod-end'));

        });

} else {

    $('li.prod').toggle(false);

    $('li.' + $(this).text().replace(/ /g, "-")).toggle(true);

        $('li.' + ($(this).text().replace(/ /g, "-")) + ':visible').each(function(i){

            i && ((i+1)%4 || $(this).addClass('prod-end'));

        });

}

});

HTML

<div class="subNav">
<ul>
<li class="button active" id="all">all</li>
<li class="button" id="swimming">swimming</li>
<li class="button" id="lawn">lawn games</li>
</ul>
</div>
<ul>
<li class="prod swimming"></li>
<li class="prod swimming"></li>
<li class="prod lawn-games"></li>
<li class="prod lawn-games"></li>
<li class="prod lawn-games"></li>
</ul>

Answer 1

Hey found a better way to solve this. I just used the first letter of the text rather than trying to replace the white spaces. Simpler solution.

Thanks for the help.