Stop using and deprecate jquery.delayedBind
authorBartosz Dziewoński <matma.rex@gmail.com>
Sat, 22 Feb 2014 19:19:49 +0000 (20:19 +0100)
committerBartosz Dziewoński <matma.rex@gmail.com>
Sat, 22 Feb 2014 19:23:26 +0000 (19:23 +0000)
commit86947ae9aa0e3065542fe0993db724cf515a78ad
treec08e49cd6e7123a64d7c12c6c452aa8e9c713242
parentee3945f26ba8b5e1c30a3704426235d16b1b8fa7
Stop using and deprecate jquery.delayedBind

Reasons:
* It provides the same functionality as jquery.throttle-debounce, but
  in a hackier and less flexible way
* It's (to my knowledge) not used outside of core, while
  jquery.throttle-debounce is - deciding on one can lower the payload
  size a little bit
* It's a custom library and we have too many of those

Only two modules in core were using it:
* jquery.expandableField: It was, in fact, used incorrectly, the code
  needs a simple setTimeout / clearTimeout pair with no debouncing.
  The bug made it possible to keep focus on a field while it was
  unexpanded (by quickly triggering blur and focus events in order).
* skins.vector.js: Straightforwardly converted the usage to a
  $.debounce call. Also fixed a bug where the window resize handler
  was bound for each $.fn.collapsibleTabs call instead of once.

The module will be removed in MediaWiki 1.24:
Ifc84b09a78007a6a0ea5676b0f12a38937dca2e7.

Change-Id: I83ba37a9568a171d9f3654f6bfdb6064e0e65bd4
RELEASE-NOTES-1.23
resources/Resources.php
resources/jquery/jquery.delayedBind.js
resources/jquery/jquery.expandableField.js
skins/vector/collapsibleTabs.js