library-mirrors/jquery-ui
1
0
Fork 0
mirror of https://github.com/jquery/jquery-ui.git synced 2024-11-21 11:04:24 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Accordion: Fixed destroy method with new markup.

Browse Source
This commit is contained in:
Scott González 2009-01-30 03:27:44 +00:00
parent 902c1910e7
commit 967d6595a6
3 changed files with 49 additions and 57 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
tests/unit/accordion/accordion.html
View File

@ -19,7 +19,6 @@
<div id="main">
<div id="list1">
<div>
<a>There is one obvious advantage:</a>
<div>
<p>
@ -30,8 +29,6 @@
Well, at least no free beer. Perhaps a bear, if you can afford it.
</p>
</div>
</div>
<div>
<a>Now that you've got...</a>
<div>
<p>
@ -43,8 +40,6 @@
We could talk about renting one.
</p>
</div>
</div>
<div>
<a>Rent one bear, ...</a>
<div>
<p>
@ -55,11 +50,10 @@
</p>
</div>
</div>
</div>
<ul id="navigation">
<li><a href="?p=1.1.1">Guitar</a></li>
<li>
<a href="?p=1.1.1">Guitar</a>
<ul>
<li><a href="?p=1.1.1.1">Electric</a></li>
<li><a href="?p=1.1.1.2">Acoustic</a></li>
@ -68,8 +62,8 @@
<li><a href="?p=1.1.1.5">Accessories</a></li>
</ul>
</li>
<li><a href="?p=1.1.2"><span>Bass</span></a></li>
<li>
<a href="?p=1.1.2"><span>Bass</span></a>
<ul>
<li><a href="?p=1.1.2.1">Electric</a></li>
<li><a href="?p=1.1.2.2">Acoustic</a></li>
@ -80,8 +74,8 @@
<li><a href="?p=1.1.2.5">Accessories</a></li>
</ul>
</li>
<li><a href="?p=1.1.3">Drums</a></li>
<li>
<a href="?p=1.1.3">Drums</a>
<ul>
<li><a href="?p=1.1.3.2">Acoustic Drums</a></li>
<li><a href="?p=1.1.3.3">Electronic Drums</a></li>

30
tests/unit/accordion/accordion.js
View File

@ -24,13 +24,13 @@ test("basics", function() {
test("autoHeight", function() {
$('#navigation').accordion({ autoHeight: false });
equals( 90, $('#navigation div:first').height() );
equals( 126, $('#navigation div:eq(1)').height() );
equals( 54, $('#navigation div:last').height() );
equals( $('#navigation > li:eq(1)').height(), 90 );
equals( $('#navigation > li:eq(3)').height(), 126 );
equals( $('#navigation > li:eq(5)').height(), 54 );
$('#navigation').accordion("destroy").accordion({ autoHeight: true });
equals( 126, $('#navigation div:first').height() );
equals( 126, $('#navigation div:eq(1)').height() );
equals( 126, $('#navigation div:last').height() );
equals( $('#navigation > li:eq(1)').height(), 126 );
equals( $('#navigation > li:eq(3)').height(), 126 );
equals( $('#navigation > li:eq(5)').height(), 126 );
});
test("activate, numeric", function() {
@ -94,7 +94,7 @@ function state2(accordion) {
}
test("handle click on header-descendant", function() {
var ac = $('#navigation').accordion({ autoHeight: false })
var ac = $('#navigation').accordion({ autoHeight: false });
ac.triggerEvent("click", $('#navigation span:contains(Bass)')[0]);
state2(ac, 0, 1, 0);
});
@ -114,17 +114,17 @@ test("accordionchange event, open closed and close again", function() {
alwaysOpen: false
})
.one("accordionchange", function(event, ui) {
equals( ui.oldHeader.size(), 0 )
equals( ui.oldContent.size(), 0 )
equals( ui.newHeader.size(), 1 )
equals( ui.newContent.size(), 1 )
equals( ui.oldHeader.size(), 0 );
equals( ui.oldContent.size(), 0 );
equals( ui.newHeader.size(), 1 );
equals( ui.newContent.size(), 1 );
})
.accordion("activate", 0)
.one("accordionchange", function(event, ui) {
equals( ui.oldHeader.size(), 1 )
equals( ui.oldContent.size(), 1 )
equals( ui.newHeader.size(), 0 )
equals( ui.newContent.size(), 0 )
equals( ui.oldHeader.size(), 1 );
equals( ui.oldContent.size(), 1 );
equals( ui.newHeader.size(), 0 );
equals( ui.newContent.size(), 0 );
})
.accordion("activate", 0);
});

4
ui/ui.accordion.js
View File

@ -105,9 +105,7 @@ $.widget("ui.accordion", {
this.headers.find("a").removeAttr("tabindex");
this.headers.children(".ui-icon").remove();
this.headers.next().removeClass("ui-accordion-content ui-accordion-content-active").each(function() {
$(this).parent().replaceWith(this);
});
this.headers.next().removeClass("ui-accordion-content ui-accordion-content-active");
},