MediaWiki talk:Gadget-exlinks.js

Update

This edit request has been answered. Set the |answered= parameter to no to reactivate your request.

Hi!

Could someone update this gadget to use jQuery and the recommended functions from MW 1.17? It would be something like this:

$( function() {
	var $alinks = $( '#content' ).find( 'a' );
	$alinks.each( function() {
		var $tablink = $( this );
		if ( $tablink.hasClass( 'external' ) && $tablink.attr( 'href' ).indexOf( mw.config.get( 'wgServer' ) ) !== 0 ) {
			$tablink.attr( 'target', '_blank' );
		}
	});
});

After the change, I think it will be possible to change the definition of this gadget to

* exlinks[ResourceLoader|dependencies=mediawiki.util]|exlinks.js

Helder 18:09, 27 July 2011 (UTC)[reply]

Is that not required to be inside a document.ready ? —TheDJ (talk • contribs) 22:04, 31 July 2011 (UTC)[reply]

Yep! And it is.

The code $(handler) is equivalent to $(document).ready(handler). Helder 23:22, 31 July 2011 (UTC)[reply]

This does not work. mw.util.$content is null for me. (Is init'ed in a .ready() after mw.util is considered to be init'ed....... —TheDJ (talk • contribs) 21:39, 1 August 2011 (UTC)[reply]

That is weird. On Portuguese Wikibooks it works fine. Besides, if I copy the following to the address bar (here, on English Wikipedia):

javascript:alert( mw.util.$content )

I get "[object Object]", and not null. Helder 12:31, 2 August 2011 (UTC)[reply]

Just replace var $alinks = mw.util.$content.find( 'a' ); with var $alinks = $( '#content' ).find( 'a' );. — Edokter (talk) — 20:01, 3 August 2011 (UTC)[reply]

I'll ask Edokter to make this change himself, as he seems to know what this is all about. — Martin (MSGJ · talk) 13:53, 12 September 2011 (UTC)[reply]

Changed the code above, but not making it live before it is tested. — Edokter (talk) — 21:25, 12 September 2011 (UTC)[reply]

If TheDJ is using Modern skin that would explain why mw.util.$content doesn't work for him, see mw:Talk:ResourceLoader/Default_modules#mw.util.$content undefined in Modern. — AlexSm 00:54, 13 September 2011 (UTC)[reply]

Can this get deployed with MW 1.18? --Locos epraix ~ Beastepraix 15:10, 1 October 2011 (UTC)[reply]