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 comment:1
| component: | unfiled → support |
|---|---|
| milestone: | None → 1.9 |
| priority: | undecided → low |
Changed September 19, 2012 02:19PM UTC by comment:2
| status: | new → open |
|---|
Changed October 17, 2012 07:06PM UTC by comment:3
| resolution: | → fixed |
|---|---|
| status: | open → closed |
Fixes #12569. Improve Feature Detect For oldIE bubbling. closes gh-967
Changeset: 063ea024e945adbeefbf88578f8ba424ca8a0939
Changed November 08, 2012 10:10PM UTC by 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 comment:5
| resolution: | fixed |
|---|---|
| status: | closed → reopened |
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 comment:6
| owner: | → dmethvin |
|---|---|
| priority: | low → high |
| status: | reopened → assigned |
Changed November 15, 2012 02:21AM UTC by comment:7
| resolution: | → fixed |
|---|---|
| status: | assigned → closed |
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 comment:8
| resolution: | fixed |
|---|---|
| status: | closed → reopened |
Changed November 19, 2012 05:43PM UTC by comment:9
| status: | reopened → assigned |
|---|
Changed December 21, 2012 03:27AM UTC by comment:10
| resolution: | → fixed |
|---|---|
| status: | assigned → closed |
Fix #12569. Improve feature detect for event bubbling. Close gh-1076.
Changeset: c8c6ab6924b48832bfc2b94d40887b2f1b6c891e
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.