Ticket #6126 (closed feature: fixed)

Opened 4 years ago

Last modified 2 years ago

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:
Blocking: Blocked by:

Description

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

Change History

comment:1 Changed 4 years ago by scott.gonzalez

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

Fixed in  a2ddfd5.

comment:2 Changed 3 years ago by scott.gonzalez

  • Milestone changed from 1.9 to 1.8.6

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

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

Changeset: a2ddfd51072952969792cd3b503e573ea8f18d30

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