Ticket #7186 (closed bug: fixed)

Opened 4 years ago

Last modified 2 years ago

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:
Blocking: Blocked by:

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

comment:1 Changed 4 years ago by scott.gonzalez

  • Priority changed from minor to major
  • Status changed from new to open

comment:2 Changed 4 years ago by scott.gonzalez

comment:3 Changed 3 years ago by gnarf

  • Owner set to gnarf
  • Status changed from open to assigned

comment:4 Changed 3 years ago by gnarf

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/

comment:5 Changed 2 years ago by scott.gonzalez

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 2 years ago by scott.gonzalez

  • Status changed from assigned to closed
  • Resolution set to fixed
  • Milestone changed from 1.9.0 to 1.8.11

This is fixed in jQuery core.

Note: See TracTickets for help on using tickets.