Skip to content

Commit 10c873a

Browse files
committed
Accordion: Add panel in addition to content for create event. Fixes #8998 - Accordion: Create event uses content instead of panel.
1 parent 673c655 commit 10c873a

File tree

2 files changed

+7
-1
lines changed

2 files changed

+7
-1
lines changed

tests/unit/accordion/accordion_events.js

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ var setupTeardown = TestHelpers.accordion.setupTeardown,
66
module( "accordion: events", setupTeardown() );
77

88
test( "create", function() {
9-
expect( 10 );
9+
expect( 15 );
1010

1111
var element = $( "#list1" ),
1212
headers = element.children( "h3" ),
@@ -16,6 +16,8 @@ test( "create", function() {
1616
create: function( event, ui ) {
1717
equal( ui.header.length, 1, "header length" );
1818
strictEqual( ui.header[ 0 ], headers[ 0 ], "header" );
19+
equal( ui.panel.length, 1, "panel length" );
20+
strictEqual( ui.panel[ 0 ], contents[ 0 ], "panel" );
1921
equal( ui.content.length, 1, "content length" );
2022
strictEqual( ui.content[ 0 ], contents[ 0 ], "content" );
2123
}
@@ -27,6 +29,8 @@ test( "create", function() {
2729
create: function( event, ui ) {
2830
equal( ui.header.length, 1, "header length" );
2931
strictEqual( ui.header[ 0 ], headers[ 2 ], "header" );
32+
equal( ui.panel.length, 1, "panel length" );
33+
strictEqual( ui.panel[ 0 ], contents[ 2 ], "panel" );
3034
equal( ui.content.length, 1, "content length" );
3135
strictEqual( ui.content[ 0 ], contents[ 2 ], "content" );
3236
}
@@ -38,6 +42,7 @@ test( "create", function() {
3842
collapsible: true,
3943
create: function( event, ui ) {
4044
equal( ui.header.length, 0, "header length" );
45+
equal( ui.panel.length, 0, "panel length" );
4146
equal( ui.content.length, 0, "content length" );
4247
}
4348
});

ui/jquery.ui.accordion.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -65,6 +65,7 @@ $.widget( "ui.accordion", {
6565
_getCreateEventData: function() {
6666
return {
6767
header: this.active,
68+
panel: !this.active.length ? $() : this.active.next(),
6869
content: !this.active.length ? $() : this.active.next()
6970
};
7071
},

0 commit comments

Comments
 (0)