[SOLUCIONADO] Encontrar elemento con una clase y quitarselo

[madman_18]

Buenas tardes!

¿Cómo podría encontrar un elemento del menú con determinada clase y quitarsela?
Veréis, mi menú consta de:

Código HTML:

<nav class="menu" id="menu">
                <ul>
                    <a href="#cooperativa"><li>Cooperativa</li></a>
                    <a href="#aceites"><li>Aceites</li></a>
                    <a href="#tienda"><li>Tienda On-line</li></a>
                    <a href="#socios"><li class="activo">Zona Privada</li></a>
                    <a href="#contacto"><li>Contacto</li></a>
                </ul>
                <div class="separator-menu"></div>
            </nav>
 

Y en el script en el que asocio el click al href soy capaz de encontrar el elemento "li" con la clase activo...pero no soy capaz de quitarselo y asignarselo al que han pulsado.

Os pongo el código:

Código Javascript:

var mi_menu = "#"+id+" a[href*=#]";
        var bloque = "#"+id;
        $(mi_menu).click(function(){
            //busco cualquier objeto que tenga la clase "activo"
            if ($(bloque).find("li").hasClass('activo')){
                // hay alguno que está con clase 'activo'
                $(bloque).find("li").hasClass('activo').removeClass('activo');
                //he probado esto de aquí pero no lo realiza :( 
            }else{
                alert('No hay');
            }            
            if (location.pathname.replace(/^\//,'') === this.pathname.replace(/^\//,'')
                   && location.hostname === this.hostname) {
                     var $target = $(this.hash);
                     $target = $target.length && $target
                     || $('[name=' + this.hash.slice(1) +']');
                     if ($target.length){                         
                       var targetOffset = $target.offset().top;
                       $('html,body').stop().animate({scrollTop: targetOffset}, 1800);
                      return false;
                     }
                   }                                       
            });
 

[alexg88]

Para quitarlo sería así:

Código Javascript:

$(bloque).find("li.activo").removeClass('activo')
 

No es necesario usar hasClass y además, por eso no te funciona el código.

$(bloque).find("li").hasClass('activo').removeClass('activo');

hasClass devuelve un booleano, no puedes llamar a removeClass después

[madman_18]

Iniciado por alexg88

Para quitarlo sería así:

Código Javascript:

$(bloque).find("li.activo").removeClass('activo')
 

No es necesario usar hasClass y además, por eso no te funciona el código.

$(bloque).find("li").hasClass('activo').removeClass('activo');

hasClass devuelve un booleano, no puedes llamar a removeClass después

Gracias! Ahora sí quito la clase

Lo único que ahora me surge es: ¿cómo le añado la clase al "li" que hay dentro del "a" para que tenga la clase 'activo'? Tendrá que ser el $(this) pero habría que añadirle algo que haga referencia al "li" (es que no he tocado mucho jQuery y aún me pierdo un poco en el trato de los objetos)

[memoadian]

Bueno para empezar la estructura de tu lista está mal

es ul li a
no ul a li

ahora si lo que quieres hacer es hacer un menu con actives y no actives XD yo hago algo muy simple

Código Javascript:

$('nav li').click(function(){
    $('nav li').removeClass("active");
    $(this).addClass("active");
});
 

ejemplo online http://jsfiddle.net/9nd4j/54/

[madman_18]

Iniciado por memoadian

Bueno para empezar la estructura de tu lista está mal

es ul li a
no ul a li

ahora si lo que quieres hacer es hacer un menu con actives y no actives XD yo hago algo muy simple

Código Javascript:

$('nav li').click(function(){
    $('nav li').removeClass("active");
    $(this).addClass("active");
});
 

ejemplo online http://jsfiddle.net/9nd4j/54/

Ostras......es verdad tío! Ya decía yo que había algo en el menú que no me cuadraba...me chocaba el código jajajajaja momento despiste.
Probaré lo que me comentas a ver si así consigo el efecto que quiero :)

[madman_18]

Creo que estoy chocando con algo.....

Con el código que me has dicho, memoadian, funciona el cambio de clase bien, pero ¿cómo le marco el evento de moverse el contenido? (lo tengo hecho con jQuery para darle el efecto de scroll suave) :S :S..

Ya es que estoy un poco mosqueado con esta web...tiene código que no es mío y repartido por 14mil sitios xD y ya ofusca un poco jeje ;)

[madman_18]

Hey!!! Al final he dado con una solución....lo que he hecho a sido aumentar de tamaño los enlaces, en vez de tanta historia buscando el li, ahora mirar si es el que han pulsado etc etc.....

Muchas gracias por las ayudas!!