Skip to main content

Bug Tracker

Side navigation

#2879 closed enhancement (fixed)

Opened May 16, 2008 04:09AM UTC

Closed April 21, 2009 07:31PM UTC

Cache the event props

Reported by: flesler Owned by: brandon
Priority: minor Milestone: 1.2.4
Component: event Version: 1.2.3
Keywords: event props array fix Cc:
Blocked by: Blocking:

Hi Brandon, I haven't benchmarked this, but there's no doubt that creating such a long string, and then splitting it requires quick a while.

So I propose caching it. I randomally use $.event.props.

By the way.. what's the reason you're copying stuff like srcElement, toElement, prevValue, etc ?

Why copy non standard attributes ?

Attachments (1)
Change History (1)

Changed April 21, 2009 07:31PM UTC by brandon comment:1

resolution: → fixed
status: newclosed

you ended up fixing this yourself. :)