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: |
Description
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 comment:1
resolution: | → fixed |
---|---|
status: | new → closed |
you ended up fixing this yourself. :)