Ticket #3086 (closed bug: fixed)

Opened 5 years ago

Last modified 5 years ago

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

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

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

comment:1 Changed 5 years ago by paul

  • Milestone changed from 1.5.2 to 1.6b

comment:2 Changed 5 years ago by scott.gonzalez

Also see #3090.

comment:3 Changed 5 years ago by scott.gonzalez

  • Status changed from new to closed
  • Resolution set to fixed

Fixed in [444].

Note: See TracTickets for help on using tickets.