Opened 15 years ago
Closed 14 years ago
#2879 closed enhancement (fixed)
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 (2)
Changed 15 years ago by
Attachment: | event-props.diff added |
---|
comment:1 Changed 14 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
Note: See
TracTickets for help on using
tickets.
you ended up fixing this yourself. :)