Opened 12 years ago
Closed 10 years ago
#7186 closed bug (fixed)
addClass seems to trigger Invalid argument in IE in jquery 1.5.1
Reported by: | bluefox | Owned by: | Corey Frang |
---|---|---|---|
Priority: | major | Milestone: | 1.8.11 |
Component: | ui.effects.core | Version: | 1.8.11 |
Keywords: | Cc: | ||
Blocked by: | Blocking: |
Description
Can be reproduced by going to http://jqueryui.com/demos/addClass/ with IE (8.0.7600.16385) and click Run Effect. A script error is generated
Invalid argument. jquery.min.js, line 16 character 79183
Might be a conflict with jQuery 1.5.1?
Change History (6)
comment:1 Changed 12 years ago by
Priority: | minor → major |
---|---|
Status: | new → open |
comment:3 Changed 12 years ago by
Owner: | set to Corey Frang |
---|---|
Status: | open → assigned |
comment:4 Changed 12 years ago by
In the meantime, while waiting for jQuery Core to catch up to this bug, you can use this cssHook to fix this error:
$.cssHooks.letterSpacing = { get: function ( elem, computed ) { var ret = parseFloat( elem.style.letterSpacing ) || 0; return ret; } };
comment:5 Changed 11 years ago by
gnarf: Any reason not to just land that cssHook in UI core? If $.cssHooks exists, define it, if not, don't do anything. That should handle all versions, right? We can land that in master and merge into 1-8-stable.
comment:6 Changed 10 years ago by
Milestone: | 1.9.0 → 1.8.11 |
---|---|
Resolution: | → fixed |
Status: | assigned → closed |
This is fixed in jQuery core.
Note: See
TracTickets for help on using
tickets.
This is caused by a change in 1.5.1: https://github.com/jquery/jquery/commit/85d9343271da85fc945bf37a604873eaf247a3a7