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. :)