[mediawiki.action.edit] Clean up and bug fixes
authorKrinkle <krinkle@users.mediawiki.org>
Tue, 28 Feb 2012 03:17:05 +0000 (03:17 +0000)
committerKrinkle <krinkle@users.mediawiki.org>
Tue, 28 Feb 2012 03:17:05 +0000 (03:17 +0000)
commit71adf36262f580e1f784fcd62087cabd54d56a4b
tree98e3a0a1fdbdd1e1a0573b37c1659848898ec184
parent85bbebc039212ce4db620afc18ae267457c0f7d4
[mediawiki.action.edit] Clean up and bug fixes
* Bugfix: Locally alias the toolbar object and replace usage of 'this' with 'toolbar'. Calls to window.addButton were failing due to it referring to the dynamic 'this' context which changes when the function is a member of a different object.
* Bugfux: Move currentFocused = $( '#wpTextbox1' ) into the dom-ready hook. When executed before dom-ready the element doesn't exist yet. r111983 did this for $toolbar, but not for currentFocused (bug 34538)

* Move onReady and isReady to the local scope (introduced recently in r111983, not used or meant to be used publicly)
* Merge onReady with the $(document).ready hook function, same purpose
* JS conventions (closure arguments, var hoisting, whitespace)

* Touch r111983, r112451, r112567
resources/mediawiki.action/mediawiki.action.edit.js