Opened 11 years ago

Closed 9 years ago

#4208 closed bug (fixed)

ui.accordion overflows over it's container when one of its divisions has a scroll bar

Reported by: margenn Owned by:
Priority: minor Milestone: 1.8.2
Component: ui.accordion Version: 1.6rc6
Keywords: accordion overflow fillSpace Cc:
Blocked by: Blocking:

Description

This happens when you put the accordion div inside a parent fixed-size div and choose "fillSpace: true" option.

This happened on firefox 3.0.6, Chrome. On IE6, the entire parent conteiner resized a few pixels

I turned around this by putting a "overflow:hidden" on parent div. There's an attachment with shows better what is happening. Tks!

Attachments (1)

accordion-bug.rar (94.3 KB) - added by margenn 11 years ago.
html file with jquery library and ui to recriate the bug.

Download all attachments as: .zip

Change History (6)

Changed 11 years ago by margenn

Attachment: accordion-bug.rar added

html file with jquery library and ui to recriate the bug.

comment:1 Changed 11 years ago by Jörn Zaefferer

Milestone: TBD1.next

comment:2 Changed 11 years ago by Jörn Zaefferer

Milestone: 1.next1.7

comment:3 Changed 11 years ago by rdworth

Milestone: 1.71.8

comment:4 Changed 11 years ago by bkilinc

I found out that the resize() function of the accordion object does not see margins. to correct this I changed outerHeight() function calls to outerHeight(true), which will include margin heights. And it worked for me;

example change

maxHeight -= $(this).outerHeight();

to

maxHeight -= $(this).outerHeight(true);

and change

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

to

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

comment:5 Changed 9 years ago by Scott González

Milestone: 1.91.8.2
Resolution: fixed
Status: newclosed

This was fixed in a previous release.

Note: See TracTickets for help on using tickets.