Skip to main content

Search and Top Navigation

#3901 closed bug (fixed)

Opened January 21, 2009 06:17PM UTC

Closed January 29, 2009 04:13AM UTC

Dialog + themeroller breaks in IE6 due to negative minHeight

Reported by: cjwilks Owned by: scottgonzalez
Priority: critical Milestone: 1.7
Component: ui.dialog Version: 1.6rc5
Keywords: Cc: mctenshi@gmail.com
Blocked by: Blocking:
Description

Description courtesy Goonoo Kim. References:

Using JQuery UI 1.6rc5 and jQuery 1.3: The following example doesn't work in IE6.

#!xml
<!-- height of content is over than 150 -->
<div id="dialog">
 <p>blah blah</p>
<p>blah blah</p>
<p>blah blah</p>
<p>blah blah</p>
<p>blah blah</p>
<p>blah blah</p>
<p>blah blah</p>
<p>blah blah</p>
<p>blah blah</p>
</div>

<script type="text/javascript">
$("#dialog").dialog({minHeight: 150}); // minHeight as default
</script>

In above case, 454 Line of jQuery UI Dialog 1.6rc5 got error on IE6.

#!java

this.element
    .css({
        //***IE breaks when this is a negative number***//
        minHeight: options.minHeight - nonContentHeight, 
        height: options.height == 'auto'
            ? 'auto'
            : options.height - nonContentHeight
    }); 

Attachments (0)
Change History (4)

Changed January 21, 2009 06:20PM UTC by cjwilks comment:1

The workaround for now is to manually set minHeight to something large enough...

#!java
$("...").dialog({ minHeight: 500 });

Changed January 28, 2009 05:52AM UTC by dndrnkrd comment:2

Just commented on this in #3930 as well; recommend that

minHeight: options.minHeight - nonContentHeight,

be changed to

minHeight: Math.max(options.minHeight - nonContentHeight,0),

to keep minHeight non-negative

Changed January 29, 2009 04:10AM UTC by scottgonzalez comment:3

owner: → scott.gonzalez
priority: majorcritical
status: newaccepted

Changed January 29, 2009 04:13AM UTC by scottgonzalez comment:4

resolution: → fixed
status: acceptedclosed

Fixed in r1845. Thanks.