Skip to main content

Bug Tracker

Side navigation

#12569 closed bug (fixed)

Opened September 19, 2012 02:13PM UTC

Closed December 21, 2012 03:27AM UTC

Improve feature detect for oldIE event bubbling

Reported by: dmethvin Owned by: dmethvin
Priority: high Milestone: 1.9
Component: support Version: 1.8.0
Keywords: Cc:
Blocked by: Blocking:
Description

From a Wikipedia edit:

For example, at least some of these scripts, including the latest at the time of this writing (see line 157 for example), base a critical decision regarding event bubbling on the presence of what is assumed to be an [[Internet Explorer]] host method. Code comments in the immediate vicinity of the detection admit it is a browser sniff and confirm that the authors are trying to target [[Internet Explorer]]. In reality, the method is found in other browsers (e.g. Opera).

A check for addEventListener should be sufficient.

Attachments (0)
Change History (10)

Changed September 19, 2012 02:19PM UTC by dmethvin comment:1

component: unfiledsupport
milestone: None1.9
priority: undecidedlow

Looks like some very opinionated jQuery hater got into the wiki: http://en.wikipedia.org/w/index.php?title=JQuery&diff=506984043&oldid=506975105

See the other comments by that IP on Wikipedia.

In any case, the appropriate place to report bugs is on the bug tracker, not by sniping on our Wikipedia entry.

Changed September 19, 2012 02:19PM UTC by dmethvin comment:2

status: newopen

Changed October 17, 2012 07:06PM UTC by Merrifield, Jay comment:3

resolution: → fixed
status: openclosed

Fixes #12569. Improve Feature Detect For oldIE bubbling. closes gh-967

Changeset: 063ea024e945adbeefbf88578f8ba424ca8a0939

Changed November 08, 2012 10:10PM UTC by scottgonzalez comment:4

The multi-browser claim is complete BS. We're not trying to patch theoretical problems that may occur in a variety of non-standard event systems. We're trying to handle a case that is known to only exist with a specific non-standard event system. The old check is perfectly legitimate. We should revert this "fix" and close this as wontfix.

Changed November 09, 2012 02:38PM UTC by dmethvin comment:5

resolution: fixed
status: closedreopened

Actually, the old and new code is broken. The UI test that broke was because jQuery.support.focusinBubbles is set to false by default. We're skipping the code to dispute that claim unless we're on IE. But recent Chrome does have a focusin event and we're shimming something we shouldn't shim--I haven't looked to see what the effects of that are:

http://jsfiddle.net/dmethvin/NrKnF/

http://jsfiddle.net/dmethvin/2PdMu/

The reason we skip this test in the first place is that that setting inline handler attributes triggers errors when CSP features are enabled, and that affects Chrome.

Changed November 13, 2012 03:20PM UTC by dmethvin comment:6

owner: → dmethvin
priority: lowhigh
status: reopenedassigned

Changed November 15, 2012 02:21AM UTC by Dave Methvin comment:7

resolution: → fixed
status: assignedclosed

Revert "Fixes #12569. Improve Feature Detect For oldIE bubbling. closes gh-967"

This reverts commit 063ea024e945adbeefbf88578f8ba424ca8a0939.

I've beaten on this for a while and can't find a suitable feature detect that catches Chrome's support for focusin.

Changeset: 05fb5de3ee1ea0610e95d2081aaab8702be651de

Changed November 15, 2012 02:23AM UTC by dmethvin comment:8

resolution: fixed
status: closedreopened

Changed November 19, 2012 05:43PM UTC by timmywil comment:9

status: reopenedassigned

Changed December 21, 2012 03:27AM UTC by Oleg comment:10

resolution: → fixed
status: assignedclosed

Fix #12569. Improve feature detect for event bubbling. Close gh-1076.

Changeset: c8c6ab6924b48832bfc2b94d40887b2f1b6c891e