Author: john
Description:
Right now there is no jQuery compatible replacement for /(inject|remove)Spinner/
Do it right now!
Version: unspecified
Severity: enhancement
• bzimport | |
Aug 22 2011, 4:00 AM |
F8053: b30499.patch | |
Nov 21 2014, 11:49 PM |
Author: john
Description:
Right now there is no jQuery compatible replacement for /(inject|remove)Spinner/
Do it right now!
Version: unspecified
Severity: enhancement
Status | Subtype | Assigned | Task | ||
---|---|---|---|---|---|
Resolved | None | T31713 Extensions that need resource loader-ifying (tracking) | |||
Resolved | None | T31714 Resource Loader-ify AbuseFilter | |||
Resolved | None | T32499 Create jQuery replacements for injectSpinner and removeSpinner |
john wrote:
Spinner plugin
Add a spinner after an element
$( '#mw-abusefilter-syntaxcheck' ).injectSpinner( 'abusefilter-syntaxcheck' );
Create a spinner element and return it
$.createSpinner( 'abusefilter-syntaxcheck' );
Remove a spinner element
$.removeSpinner( 'abusefilter-syntaxcheck' );
Attached:
So FYI there are already jquery spinner plugins in MwEmbed and UploadWizard.
Can we just pick one? :)
john wrote:
Applied in r95445
To be honest I did look at those 2 implementations but I didn't feel they were general enough, and were both sort of odd. This is a direct replacement for injectSpinner found in wikibits.js, and will aid in developers who are making a transition from old style JS to use jQuery.