Skip to main content

Bug Tracker

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 anonymous comment:1

Changed June 01, 2011 05:46PM UTC by dmethvin comment:2

Changed June 01, 2011 05:51PM UTC by dmethvin comment:3

component: unfiledevent
priority: undecidedlow

Changed June 14, 2011 07:42PM UTC by rwldrn comment:4

resolution: → fixed
status: openclosed

Landing pull request 397. withinElement rewrite in event. Fixes #6234, #9357, #9447.

More Details:

Changeset: 6926247bf441deaa0441872849bb3786c257a4cf

Changed June 14, 2012 02:16AM UTC by anonymous comment:5

Pull Request jeremy scott uk