Opened 13 years ago

Closed 13 years ago

Last modified 13 years ago

#7588 closed defect (fixed)

AccordionContainer doesnt sanity check _getTargetHeight

Reported by: Sam Foster Owned by: Sam Foster
Priority: high Milestone: 1.2
Component: Dijit Version: 1.2beta
Keywords: Cc: Dustin Machi
Blocked By: Blocking:


If the accordioncontainer has 0 or small height, _getTargetHeight can be negative, which is bad (setting a height/width style to a negative px value yields a 'invalid argument' error in IE)

Change History (2)

comment:1 Changed 13 years ago by Sam Foster

Resolution: fixed
Status: newclosed

(In [15116]) Added a Math.min around _getTargetHeight's value to ensure it returns at least 0 fixes #7588

comment:2 Changed 13 years ago by bill

Milestone: tbd1.2
Note: See TracTickets for help on using tickets.