Opened 10 years ago

Closed 10 years ago

#3086 closed bug (fixed)

size() in dialog buggy under IE with margin values 'auto'

Reported by: doyer.guyllaume@… Owned by: Scott González
Priority: blocker Milestone: 1.6b
Component: ui.dialog Version: 1.5.1
Keywords: dialog IE size margin value auto Cc:
Blocked by: Blocking:

Description

dialog is buggy with Internet Explorer 6 & 7 when container margin value are set to "auto", due to an NaN value for "lrMargin" in the size() function.

I just end up with the following temp fix, waiting for this bug to be fixed in the next realease: 

tbMargin = parseInt( content.css('margin-top') != "auto" ? content.css('margin-top') : 0,10) + parseInt( content.css('margin-bottom') != "auto" ? content.css('margin-bottom') : 0,10),
lrMargin = parseInt( content.css('margin-left') != "auto" ? content.css('margin-left') : 0,10) + parseInt( content.css('margin-right') != "auto" ? content.css('margin-right') : 0,10);

instead of: 

tbMargin = parseInt(content.css('margin-top'),10) + parseInt(content.css('margin-bottom'),10),
lrMargin = parseInt(content.css('margin-left'),10) + parseInt(content.css('margin-right'),10);

Change History (3)

comment:1 Changed 10 years ago by paul

Milestone: 1.5.2 → 1.6b

comment:2 Changed 10 years ago by Scott González

Also see #3090.

comment:3 Changed 10 years ago by Scott González

Resolution: → fixed
Status: new → closed

Fixed in [444].

Note: See TracTickets for help on using tickets.