Side navigation
#9447 closed enhancement (fixed)
Opened May 27, 2011 06:20PM UTC
Closed June 14, 2011 07:42PM UTC
Last modified June 14, 2012 02:16AM UTC
Modify withinElement() to use $.contains
| Reported by: | brandon@palladiumconsulting.com | Owned by: | |
|---|---|---|---|
| Priority: | low | Milestone: | 1.next |
| Component: | event | Version: | 1.6.1 |
| Keywords: | Cc: | ||
| Blocked by: | Blocking: |
Description
This method currently performs its own search to see if one element is inside another. It should use $.contains instead, reducing code size and (possibly) being faster since $.contains uses native methods.
Attachments (0)
Change History (5)
Changed May 27, 2011 07:48PM UTC by comment:1
Changed June 01, 2011 05:46PM UTC by comment:2
| status: | new → open |
|---|
Changed June 01, 2011 05:51PM UTC by comment:3
| component: | unfiled → event |
|---|---|
| priority: | undecided → low |
Changed June 14, 2011 07:42PM UTC by comment:4
| resolution: | → fixed |
|---|---|
| status: | open → closed |
Landing pull request 397. withinElement rewrite in event. Fixes #6234, #9357, #9447.
More Details:
- https://github.com/jquery/jquery/pull/397
- http://bugs.jquery.com/ticket/6234
- http://bugs.jquery.com/ticket/9357
- http://bugs.jquery.com/ticket/9447
Changeset: 6926247bf441deaa0441872849bb3786c257a4cf
Changed June 14, 2012 02:16AM UTC by comment:5
Pull Request jeremy scott uk
Pull Request: https://github.com/jquery/jquery/pull/396