MediaWiki talk:Gadget-morebits.js
Latest comment: 14 years ago by Amalthea in topic IndexOf x $.inArray
 | Before modifying or proposing changes to this script, please note: Modifications made to this gadget script are likely to be unintentionally overwritten, as the script is automatically synchronized with an off-wiki code repository. Please notify Twinkle's maintainers of any changes you propose or make to this script. |
 | This edit request has been answered. Set the |answered= parameter to no to reactivate your request. |
Could a friendly admin please copy the contents of User:This, that and the other/temp/MediaWiki:Gadget-morebits.js to this page? I am currently in the process of testing this gadget script. — This, that, and the other (talk) 08:09, 16 May 2011 (UTC)
- Friendly admin gladly complies. Amalthea 10:10, 16 May 2011 (UTC)
IndexOf x $.inArray
| | This edit request has been answered. Set the |answered= parameter to no to reactivate your request. |
Wouldn't be better to change
mw.config.get( 'wgUserGroups' ).indexOf( group ) !== -1
to
$.inArray(group, mw.config.get( 'wgUserGroups' ))
to ensure cross-browser support? Helder 13:55, 18 May 2011 (UTC)
- I have no idea. But as there is no opposition I have made the change for you. — Martin (MSGJ · talk) 10:14, 19 May 2011 (UTC)
- Fair enough. Personally, I think it's a performance decrease, and doesn't really improve legibility. But, that aside, any changes to this page are likely to get unintentionally overwritten, unless Twinkle maintainers get a heads-up at WT:TW. — This, that, and the other (talk) 10:40, 19 May 2011 (UTC)
- Helder's point was mainly browser compatibility, I think: IE8 doesn't support JS 1.6 and thus doesn't know Array.indexOf.
Don't know how many other problems morebits has in IE8 (and older), of course. Amalthea 11:03, 19 May 2011 (UTC)
- Helder's point was mainly browser compatibility, I think: IE8 doesn't support JS 1.6 and thus doesn't know Array.indexOf.
- Fair enough. Personally, I think it's a performance decrease, and doesn't really improve legibility. But, that aside, any changes to this page are likely to get unintentionally overwritten, unless Twinkle maintainers get a heads-up at WT:TW. — This, that, and the other (talk) 10:40, 19 May 2011 (UTC)
| | This edit request has been answered. Set the |answered= parameter to no to reactivate your request. |
Actually, that change broke stuff. Could an admin please revert it? Thanks, — This, that, and the other (talk) 10:57, 19 May 2011 (UTC)