Bug Tracker

Opened 12 years ago

Closed 11 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)

event-props.diff (1.2 KB) - added by flesler 12 years ago.

Download all attachments as: .zip

Change History (2)

Changed 12 years ago by flesler

Attachment: event-props.diff added

comment:1 Changed 11 years ago by brandon

Resolution: fixed
Status: newclosed

you ended up fixing this yourself. :)

Note: See TracTickets for help on using tickets.