Skip to main content

Search and Top Navigation

#6126 closed feature (fixed)

Opened September 30, 2010 12:22AM UTC

Closed September 30, 2010 12:38AM UTC

Last modified April 12, 2012 04:41PM UTC

Widget: Add create event

Reported by: scottgonzalez 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.

Attachments (0)
Change History (4)

Changed September 30, 2010 12:38AM UTC by scottgonzalez comment:1

resolution: → fixed
status: newclosed

Fixed in a2ddfd5.

Changed October 25, 2010 06:54PM UTC by scottgonzalez comment:2

milestone: 1.91.8.6

Changed November 19, 2010 06:26PM UTC by Scott González comment:3

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

Changeset: a2ddfd51072952969792cd3b503e573ea8f18d30

Changed April 12, 2012 04:41PM UTC by olejorgenb comment:4

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.