Opened 13 years ago

Closed 13 years ago

Last modified 11 years ago

#6126 closed feature (fixed)

Widget: Add create event

Reported by: Scott González Owned by:
Priority: major Milestone: 1.8.6
Component: ui.widget Version: 1.8.5
Keywords: Cc:
Blocked by: Blocking:

Description

Widgets should trigger a create event so extensions can bind to an event rather than needing to proxy the _create method.

Change History (4)

comment:1 Changed 13 years ago by Scott González

Resolution: fixed
Status: newclosed

Fixed in a2ddfd5.

comment:2 Changed 12 years ago by Scott González

Milestone: 1.91.8.6

comment:3 Changed 12 years ago by Scott González

Widget: Added create event. Fixes #6126 - Widget: Add create event.

Changeset: a2ddfd51072952969792cd3b503e573ea8f18d30

comment:4 Changed 11 years ago by olejorgenb

I'm sure this is useful in some cases, but afaics it's not used at all in the ui library (yet).

On some pages with lots of widgets (eg. buttons) the trigger takes significant time to run.

On a syntetic "benchmark" http://jsfiddle.net/2cAVn/4/ using chromes profiler trigger("create") takes 11% of the button creation time.

On a real life example I'm up to 33% (more complicated dom probably)

It's not a problem on beefy machines running chrome, but it's significant on older ones using ie8

Hopefully this can be considered.

Note: See TracTickets for help on using tickets.