Bug Tracker

Opened 7 years ago

Closed 7 years ago

Last modified 5 years ago

#5851 closed bug (fixed)

IE8 does not support the change event while using live

Reported by: matiasnu Owned by: brandon
Priority: major Milestone: 1.4.2
Component: event Version: 1.4.1
Keywords: Cc:
Blocked by: Blocking:

Description

According to http://jquery14.com/day-01/jquery-14

The change and submit events work reliably across browsers for both normal and live events. We override the normal change and submit events in Internet Explorer and replace them with events that work identically to the other browsers.

---

This is not true for IE8, which seems to ignore live bound change events. (is I use bind, it works as advertised, though)

Change History (10)

comment:1 Changed 7 years ago by matiasnu

This text should also be changed until the bug is fixed; --- All Events Can Be Live Events

With the exception of ready, focus (use focusin instead), and blur (use focusout instead), all events that can be bound using .bind() can also be live events.

We’re very proud to count some addtional events amongst those supported by live(). 1.4 introduces cross-browser support for change, submit, focusin, focusout, mouseenter, and mouseleave via the event delegation in .live().

comment:2 Changed 7 years ago by simonjefford

I'm seeing the same thing here. Support for all events in live() across all the major browsers was the reason we just upgraded.

comment:3 Changed 7 years ago by trixi

I have already posted a possible fix in the old discussion-group:

I have looked into this. The "trigger-line" in the testChange-function has to be changed from this:

return jQuery.event.trigger( e, arguments[1], this );

to this:

return jQuery.event.trigger( e, arguments[1], elem );

Additional It seems, that there was a typo. The condition above contains the following statement:

elem.type !== "select"

wich is always true. I think this can be deleted, without problems.

Some further infos about what is causing the bug, can be also found in the post: http://groups.google.com/group/jquery-dev/browse_thread/thread/e74de6015020d160#msg_97e3d41e210ebcc8

comment:4 Changed 7 years ago by john

  • Resolution set to fixed
  • Status changed from new to closed

comment:5 Changed 7 years ago by matiasnu

  • Resolution fixed deleted
  • Status changed from closed to reopened

It seems that 1.4.1 does not detect the first change on an element in IE8 but the following changes work as expected. (attaching @simonjefford test case using 1.4.1 instead)

comment:6 Changed 7 years ago by parrots

Problem seems to be in the testChange function. It's is using jQuery.data to cache the value of the input. The first time called it tries to get the previous value and it's undefined since it hasn't been cached yet. It returns without triggering the event because of the if check that fires because it's undefined. The next time through the cache has been created for the element with it's previous value and it can evaluate the change properly.

comment:7 Changed 7 years ago by parrots

Changing the beforeactivate function to the following seems to fix this issue in IE:

var elem = e.target;

if (elem.nodeName.toLowerCase() === "input"
elem.nodeName.toLowerCase() === "textarea") {

jQuery.data(elem, "_change_data", getVal(elem));

}

comment:8 Changed 7 years ago by john

  • Resolution set to fixed
  • Status changed from reopened to closed

comment:9 Changed 7 years ago by john

  • Milestone changed from 1.4.1 to 1.4.2
  • Version changed from 1.4 to 1.4.1

comment:10 Changed 6 years ago by anonymous

This seems to be broken again in IE9

Note: See TracTickets for help on using tickets.