Skip to content

Commit

Permalink
Accordion: Remove code that sets accordion height. Fixed #8696 - Mult…
Browse files Browse the repository at this point in the history
…iple collapsible Accordions won't work together.
  • Loading branch information
Maciej Mroziński authored and scottgonzalez committed Oct 23, 2012
1 parent 8bb05d2 commit f4b3cd5
Showing 1 changed file with 0 additions and 7 deletions.
7 changes: 0 additions & 7 deletions ui/jquery.ui.accordion.js
Expand Up @@ -75,7 +75,6 @@ $.widget( "ui.accordion", {
.show();

this._createIcons();
this.originalHeight = this.element[0].style.height;
this.refresh();

// ARIA
Expand Down Expand Up @@ -198,7 +197,6 @@ $.widget( "ui.accordion", {
}
});
if ( this.options.heightStyle !== "content" ) {
this.element.css( "height", this.originalHeight );
contents.css( "height", "" );
}
},
Expand Down Expand Up @@ -289,7 +287,6 @@ $.widget( "ui.accordion", {
heightStyle = this.options.heightStyle,
parent = this.element.parent();

this.element.css( "height", this.originalHeight );

if ( heightStyle === "fill" ) {
// IE 6 treats height like minHeight, so we need to turn off overflow
Expand Down Expand Up @@ -332,10 +329,6 @@ $.widget( "ui.accordion", {
})
.height( maxHeight );
}

if ( heightStyle !== "content" ) {
this.element.height( this.element.height() );
}
},

_activate: function( index ) {
Expand Down

0 comments on commit f4b3cd5

Please sign in to comment.