2013-08-02 57 views
0

我想从父级删除“打开”类,当我关闭切换,但是我下面的代码不工作。它添加了“打开”类,但在关闭时不会删除。任何人都可以看到我要去哪里错了吗?jquery从父div中删除类

$('.sub-menu').hide(); 
$('.toggle').click(function() { 
    if (!$(".sub-menu").is(":visible")) 
     $(this).addClass("open"); 

    $(this).next('.sub-menu').slideToggle(function() { 
     if (!$(".sub-menu").is(":visible")) 
      $(this).parent('.toggle').removeClass("open"); 
    }); 

}); 

更新:“nav”的HTML在请求下面。

<dd class="toggle open"> 
    <a id="navProducts">Products</a> 
</dd> 
<div class="sub-menu" style="display: block;">  
    <dd class="sub-menu-option"> 
    <a id="navBluray" href="nhdadmin/products/bluray">Blu-ray</a> 
    </dd> 
    <dd class="sub-menu-option"> 
    <a id="navXbox" href="nhdadmin/products/xbox">Xbox</a> 
    </dd> 
    <dd class="sub-menu-option"> 
    <a id="navPlayStation" href="nhdadmin/products/playstation">PlayStation</a> 
    </dd> 

+1

你能给我们一些html的工作,请? – pythonian29033

+0

这可能是问题吗? http://jqueryminute.com/jquery-parent-vs-parents/ –

+1

对我来说,由于您使用.next()来定位.sub菜单,它不是.toggle的子元素,而是一个兄弟。但它只是猜测真的没有任何HTML – ninja

回答

1

试试这个:

$('.toggle').click(function() { 
    if (!$(".sub-menu").is(":visible")) 
     $(this).addClass("open"); 

    var currentSlide = $(this); 
    $(this).next('.sub-menu').slideToggle(function() { 
     if (!$(".sub-menu").is(":visible")) 
      currentSlide.removeClass("open"); 
    }); 
}); 
+0

工作的一种享受,谢谢。 – thatuxguy

+0

这和我的答案一样,但是有一个额外的变量。 – Trojan

+0

但我有代码,所有程序员都不会说英文。 less yada-yada,更多代码 – pythonian29033

2

您要添加的 “开放” 类上点击.toggle元素。然后尝试将其从$(this)(切换元素).parent('.toggle')(切换元素的父级“toggle” - 不存在)中删除。

删除.parent('.toggle'),您的代码将工作。

1

尝试

$('.sub-menu').hide(); 
$('.toggle').click(function() { 
    var toggle = $(this); 

    toggle.next('.sub-menu').slideToggle(function() { 
     if ($(this).is(":visible")){ 
      toggle.addClass("open"); 
     } else { 
      toggle.removeClass("open"); 
     } 
    }); 

}); 
+0

也运行良好,我选择了另一个解决方案,因为它在菜单打开之前添加了CSS,并在关闭后关闭:)谢谢。 – thatuxguy

0

试试这个

$(this).parent('.toggle').removeClass("open"); 

代替

$(this).prev('.toggle').removeClass("open"); 

希望它会帮助