jQuery 兄弟/removeClass“active”只是行不通

Question

我一直在尝试让这个箭头切换脚本在我的页面上工作。我想我错过了一些简单的东西，但我只是不知道它是什么。花了2个小时才弄清楚这个问题。

脚本：

$(document).ready(function(){

    $(".accordion h3:first").addClass("active");
    $(".accordion a:first").addClass("active");
    $(".accordion p:not(:first)").hide();

    $(".accordion h3").click(function() 
    {
        $(this).next("p").slideDown("slow")
               .siblings("p:visible").slideUp("slow");
    });

    $(".accordion a").click(function() 
    {
        $(this).css({background:"url(images/arrowdown.gif) bottom right no-repeat"});
        $(this).siblings("a").removeClass("active");
    });  
});

css：

.accordion h3 a {
    padding:20px 0 0 190px; 
    display:block;  
    background:url(images/arrowup.gif) no-repeat bottom right; }

.accordion h3 a:hover { text-decoration:none;   }

.accordion h3 a.active {
    background:url(images/arrowdown.gif) no-repeat bottom right; }

然后我的 HTML：

<div class="accordion">
    <h3 id="tab1">
        <a href="#">Title 1</a>
    </h3>
    <p>Description Title 1</p>
    <h3 id="tab2">
        <a href="#">Title 2</a>
    </h3>
    <p>Description Title 2</p>
    <h3 id="tab3">
        <a href="#">Title 3</a>
    </h3>
    <p>Description Title 3</p>
</div>

所以 up &向下箭头位于“a href”标签内，H3 标签中有不同的背景图像，具体取决于“tab”ID。我希望这是有道理的。

先感谢您！！

Answer 1

问题是 siblings 适用于同一父级下的元素。您的 a 链接不在同一父链接下，因为每个链接都包含在 h3 中。

所以我相信这就是您想要的

$(document).ready(function(){

    $(".accordion h3:first").addClass("active");
    $(".accordion a:first").addClass("active");
    $(".accordion p:not(:first)").hide();

    $(".accordion h3").click(function() {
        $(this)
            .addClass('active') //set the current as active
            .siblings("h3") //find sibling h3 elements
            .removeClass("active") // and remove the active from them
            .end() // return selection to the current element
            .next("p") // find the next p
            .slideDown("slow") // and show it
            .siblings("p:visible") // find the other visible p elements
            .slideUp("slow"); // and hide them

        $(this)
            .find('a') // find a under current element
            .addClass('active') // and add active class
            .end() // return to current element
            .siblings('h3') // find sibling h3 elements
            .find('a') // find their a elements
            .removeClass('active'); // and remove the active
    });

});

演示 http://jsfiddle.net/gaby/NSvQB/

Answer 2

该脚本中有两个错误：

1. 每个 元素都是

   元素的唯一子元素。没有其他 兄弟姐妹。您想要的是找到手风琴标题内的所有其他 元素：

   $(this).closest('.accordion').find('h3 a').removeClass("active");
   

2. Setting the style on an element ($(this).css(... );) 将覆盖任何其他样式定义。稍后通过更改类来更改背景（在本例中，删除类 active）不会产生任何效果^[演示]。因此，您应该设置 active 类，而不是直接设置样式：

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

Working 演示

---

更新：您还可以通过将所有内容放入 h3 点击事件处理程序中来简化整个代码，甚至添加 active< /code> 类。

JavaScript：

$(".accordion h3").click(function() {

    $(this).siblings().removeClass("active");
    $(this).addClass('active');

    $(this).next("p").slideDown("slow")
           .siblings("p:visible").slideUp("slow");

});

CSS（更改部分）：

.accordion h3.active a {
    background:red; 
}

DEMO 2