r103962 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r103961‎ | r103962 | r103963 >
Date:21:46, 22 November 2011
Author:brion
Status:ok
Tags:
Comment:
* (bug 32582) Fix TOC show/hide link regression on IE 8

MFT part of r93364: $('<a href=blah>') doesn't work in IE 8 without closing tag (</a> at end or <a .../>). Trunk cleanup had already fixed this.
Modified paths:
  • /branches/wmf/1.18wmf1/resources/mediawiki/mediawiki.util.js (modified) (history)

Diff [purge]

Index: branches/wmf/1.18wmf1/resources/mediawiki/mediawiki.util.js
@@ -86,13 +86,13 @@
8787 // Only add it if there is a TOC and there is no toggle added already
8888 if ( $tocContainer.size() && $tocTitle.size() && !$tocToggleLink.size() ) {
8989 var hideTocCookie = $.cookie( 'mw_hidetoc' );
90 - $tocToggleLink = $( '<a href="#" class="internal" id="togglelink">' )
 90+ $tocToggleLink = $( '<a href="#" class="internal" id="togglelink"></a>' )
9191 .text( mw.msg( 'hidetoc' ) )
9292 .click( function(e){
9393 e.preventDefault();
9494 util.toggleToc( $(this) );
9595 } );
96 - $tocTitle.append( $tocToggleLink.wrap( '<span class="toctoggle">' ).parent().prepend( '&nbsp;[' ).append( ']&nbsp;' ) );
 96+ $tocTitle.append( $tocToggleLink.wrap( '<span class="toctoggle"></span>' ).parent().prepend( '&nbsp;[' ).append( ']&nbsp;' ) );
9797
9898 if ( hideTocCookie == '1' ) {
9999 // Cookie says user want toc hidden

Past revisions this follows-up on

RevisionCommit summaryAuthorDate
r93364Applying code conventions:...krinkle05:53, 28 July 2011
r103961* (bug 32582) Fix TOC show/hide link regression on IE 8...brion21:44, 22 November 2011

Status & tagging log