Opened 12 years ago

Closed 12 years ago

#2915 closed bug (fixed)

[effects-core] in createWrapper, check if top and left are NaN with isNaN()

Reported by: lrbabe Owned by: paul
Priority: major Milestone: 1.5
Component: ui.effects.core Version: 1.5b4
Keywords: effects, createWrapper, isNaN Cc:
Blocked by: Blocking:

Description

Before setting the wrapper css, createWrapper check whether top and left are number with if (top.constructor != Number) which cause problems with IE browsers We should use if (isNaN(top)) instead

Attachments (1)

patch.txt​ (909 bytes) - added by lrbabe 12 years ago.
patch to replace the test with isNaN()

Download all attachments as: .zip

Change History (3)

Changed 12 years ago by lrbabe

Attachment: patch.txt​ added

patch to replace the test with isNaN()

comment:1 Changed 12 years ago by paul

Component: ui.core → effects.core

comment:2 Changed 12 years ago by paul

Resolution: → fixed
Status: new → closed

Thanks, fixed!

Note: See TracTickets for help on using tickets.