IE8 : Non valid argument at line 4618

Reported by:	christian	Owned by:
Priority:		Milestone:	1.4.2
Component:	unfiled	Version:	1.4.2
Keywords:		Cc:
Blocked by:		Blocking:

Description

Both in the minified 1.4.2 or in the dev release I got this error with IE8 (XP SP3)

Invalid argument (at line 4618 for the dev version)

Attachments (0)

Change History (4)

Changed March 16, 2010 02:36PM UTC by smdavidson comment:1

I modified line 4586 to fix this one...

old:

if ( (name === "width" || name === "height") && parseFloat(value) < 0) {

new:

if ( (name === "width" || name === "height") && (parseFloat(value) < 0 || parseFloat(value) + "" === "NaN")) {

Changed March 16, 2010 03:25PM UTC by smdavidson comment:2

Addendum to the previous post...there should be an 'OR' operator between parseFloat(value) < 0 and parseFloat(value + "" === "NaN". I think the text editor removed the pipes.

Changed March 18, 2010 12:42AM UTC by dmethvin comment:3

resolution:	→ worksforme
status:	new → closed

It sounds like you may be passing invalid arguments to jQuery. Please provide a simple test case that shows how the problem occurs.

Changed March 15, 2011 11:09AM UTC by psd-dude.com comment:4

Hi,

might I suggest adding to line 4586 the following


old:
if ( (name === "width" || name === "height")

new:

if ( (name === "width" || name === "height" || name === "marginLeft" || name === "marginRight" || name === "marginTop" || name === "marginBottom")

I bumped into the error signaled and your fix did not work for me in IE8. I was using the jqueryUI .tabs and the error was caused by the attempt to set "marginLeft" with "NaNpx"

hope this helps

kindest regards

John

Side navigation