jenkins-bot has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/307686 )
Change subject: Apply MediaWiki coding conventions for JavaScript ...................................................................... Apply MediaWiki coding conventions for JavaScript * Add closure. * Use $foo.find( '...' ) instead of $( '...', $foo ). * Use $( '<span>' ) instead of $( '<span />' ). * Remove not necessary public object mw.tocTree. * Remove not necessary return true. Change-Id: I99f9a11154b5e49cdb60e0d079f4d86070cc2666 --- M modules/ext.toctree.js 1 file changed, 14 insertions(+), 18 deletions(-) Approvals: Krinkle: Looks good to me, approved jenkins-bot: Verified diff --git a/modules/ext.toctree.js b/modules/ext.toctree.js index b091e10..097a363 100644 --- a/modules/ext.toctree.js +++ b/modules/ext.toctree.js @@ -11,9 +11,9 @@ * @licence GNU General Public Licence 2.0 or later */ -mw.tocTree = { - processClickEvent: function ( event ) { - var $ul = $( 'ul', $( this ).parent().parent() ); +( function ( mw, $ ) { + function processClickEvent( event ) { + var $ul = $( this ).parent().parent().find( 'ul' ); $ul.toggle(); if ( $ul.is( ':visible' ) ) { @@ -25,9 +25,9 @@ .text( '+' ) .attr( 'title', mw.msg( 'showtoc' ) ); } - }, + } - init: function() { + function init() { var $toc = $( '#toc' ); if ( $toc.length > 0 ) { @@ -36,20 +36,20 @@ } $toc.attr( 'cellspacing', 0 ); - var $mainUl = $( 'ul:first', $toc ); - var $mainList = $( 'li', $toc ); + var $mainUl = $toc.find( 'ul:first' ); + var $mainList = $toc.find( 'li' ); - $mainList.each( function( i ) { + $mainList.each( function ( i ) { if ( $( this ).hasClass( 'toclevel-1' ) ) { $( this ).css( 'position', 'relative' ); - var $subList = $( 'ul', $( this ) ); + var $subList = $( this ).find( 'ul' ); if ( $subList.length > 0 ) { if ( $mainUl.length > 0 ) { $mainUl.addClass( 'tocUl' ); } - var $toggleLink = $( '<span />' ).addClass( 'toggleSymbol' ); + var $toggleLink = $( '<span>' ).addClass( 'toggleSymbol' ); if ( mw.user.options.get( 'toc-expand' ) ) { $toggleLink @@ -64,9 +64,9 @@ $subList.hide(); } - $toggleLink.click( mw.tocTree.processClickEvent ); + $toggleLink.click( processClickEvent ); - var $toggleSpan = $( '<span />' ).addClass( 'toggleNode' ); + var $toggleSpan = $( '<span>' ).addClass( 'toggleNode' ); $toggleSpan.append( '[', $toggleLink, ']' ); $( this ).prepend( $toggleSpan ); @@ -74,11 +74,7 @@ } } ); } - - return true; } -}; -jQuery( function( $ ) { - mw.tocTree.init(); -} ); + $( init ); +}( mediaWiki, jQuery ) ); -- To view, visit https://gerrit.wikimedia.org/r/307686 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: I99f9a11154b5e49cdb60e0d079f4d86070cc2666 Gerrit-PatchSet: 2 Gerrit-Project: mediawiki/extensions/TocTree Gerrit-Branch: master Gerrit-Owner: Fomafix <foma...@googlemail.com> Gerrit-Reviewer: Fomafix <foma...@googlemail.com> Gerrit-Reviewer: Krinkle <krinklem...@gmail.com> Gerrit-Reviewer: jenkins-bot <> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits