#7186 closed bug (fixed)

Opened March 22, 2011 07:01PM UTC

Closed August 29, 2012 04:36PM UTC

addClass seems to trigger Invalid argument in IE in jquery 1.5.1

Reported by: bluefox Owned by: gnarf
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?

Attachments (0)
Change History (6)

Changed March 24, 2011 08:33PM UTC by scottgonzalez comment:1

priority: minor → major
status: new → open

This is caused by a change in 1.5.1: https://github.com/jquery/jquery/commit/85d9343271da85fc945bf37a604873eaf247a3a7

Changed March 24, 2011 08:57PM UTC by scottgonzalez comment:2

jQuery core bug: http://bugs.jquery.com/ticket/8627

Changed June 23, 2011 11:22AM UTC by gnarf comment:3

owner: → gnarf
status: open → assigned

Changed June 23, 2011 11:56AM UTC by gnarf comment:4

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;
    }
};

http://jsfiddle.net/gnarf/bkd4S/3/

Changed May 25, 2012 02:06AM UTC by scottgonzalez comment:5

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.

Changed August 29, 2012 04:36PM UTC by scottgonzalez comment:6

milestone: 1.9.0 → 1.8.11
resolution: → fixed
status: assigned → closed

This is fixed in jQuery core.