Opened 5 years ago

Closed 5 years ago

Last modified 4 years ago

#6126 closed feature (fixed)

Widget: Add create event

Reported by: scott.gonzalez Owned by:
Priority: major Milestone: 1.8.6
Component: ui.widget Version: 1.8.5
Keywords: Cc:
Blocked by: Blocking:


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 5 years ago by scott.gonzalez

  • Resolution set to fixed
  • Status changed from new to closed

Fixed in a2ddfd5.

comment:2 Changed 5 years ago by scott.gonzalez

  • Milestone changed from 1.9 to 1.8.6

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

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

Changeset: a2ddfd51072952969792cd3b503e573ea8f18d30

comment:4 Changed 4 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" 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.