Skip to content

Commit f4b3cd5

Browse files
Maciej Mrozińskiscottgonzalez
Maciej Mroziński
authored andcommitted
Accordion: Remove code that sets accordion height. Fixed #8696 - Multiple collapsible Accordions won't work together.
1 parent 8bb05d2 commit f4b3cd5

File tree

1 file changed

+0
-7
lines changed

1 file changed

+0
-7
lines changed

ui/jquery.ui.accordion.js

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -75,7 +75,6 @@ $.widget( "ui.accordion", {
7575
.show();
7676

7777
this._createIcons();
78-
this.originalHeight = this.element[0].style.height;
7978
this.refresh();
8079

8180
// ARIA
@@ -198,7 +197,6 @@ $.widget( "ui.accordion", {
198197
}
199198
});
200199
if ( this.options.heightStyle !== "content" ) {
201-
this.element.css( "height", this.originalHeight );
202200
contents.css( "height", "" );
203201
}
204202
},
@@ -289,7 +287,6 @@ $.widget( "ui.accordion", {
289287
heightStyle = this.options.heightStyle,
290288
parent = this.element.parent();
291289

292-
this.element.css( "height", this.originalHeight );
293290

294291
if ( heightStyle === "fill" ) {
295292
// IE 6 treats height like minHeight, so we need to turn off overflow
@@ -332,10 +329,6 @@ $.widget( "ui.accordion", {
332329
})
333330
.height( maxHeight );
334331
}
335-
336-
if ( heightStyle !== "content" ) {
337-
this.element.height( this.element.height() );
338-
}
339332
},
340333

341334
_activate: function( index ) {

0 commit comments

Comments
 (0)