resize function with autoheight

[guile]

based on version 1.7.1.

There is a problem with the resize function for autoHeight when it sets the new height based on the maxHeight. MaxHeight is calculated based on the outerHeight which is normal but then it sets the height of each element based on maxHeight. It should set the height of the element based on maxHeight -(border and padding of this resized element). Below is my bugfix proposition. I tested it and it works.

Current code from line 191:

} else if ( o.autoHeight ) {

maxHeight = 0; this.headers.next().each(function() {

maxHeight = Math.max(maxHeight, $(this).outerHeight());

}).height(maxHeight);

}

My solution: from line 191

} else if ( o.autoHeight ) {

maxHeight = 0; this.headers.next().each(function() {

maxHeight = Math.max(maxHeight, $(this).outerHeight());

}).height(function(){

return maxHeight-$(this).outerHeight()+$(this).height()

});

}

[guile]

Sorry but I made a mistake.

This is a better code

} else if ( o.autoHeight ) {

maxHeight = 0; this.headers.next().each(function() {

maxHeight = Math.max(maxHeight, $(this).outerHeight());

}).each(function() {

var newHeight=maxHeight-$(this).outerHeight()+$(this).height();
(this).height(newHeight);

});

}

[guile]

OK sorry again the mistakes above. Here it looks a lot better.

This is what I suggest from line 191:

} else if ( o.autoHeight ) {
  maxHeight = 0;
  this.headers.next().each(function() {
    maxHeight = Math.max(maxHeight, $(this).outerHeight());
  }).each(function() {
    var newHeight=maxHeight-$(this).outerHeight()+$(this).height();
    $(this).height(newHeight);
  });
}

[Jörn Zaefferer]

Fixed in r3038. The change is to just use height() instead of outerHeight(), making it much simpler then what you proposed.

Please reopen this ticket if that isn't enough for you, preferably with a usecase included.