r104182 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r104181‎ | r104182 | r104183 >
Date:16:30, 24 November 2011
Author:reedy
Status:ok
Tags:
Comment:
REL1_18 MFT r92758
Modified paths:
  • /branches/REL1_18/phase3 (modified) (history)
  • /branches/REL1_18/phase3/resources/jquery/jquery.localize.js (modified) (history)

Diff [purge]

Index: branches/REL1_18/phase3/resources/jquery/jquery.localize.js
@@ -37,7 +37,12 @@
3838 return mw.msg.apply( mw, args );
3939 };
4040 return $(this)
41 - .find( 'html\\:msg' )
 41+ // Ok, so here's the story on this selector.
 42+ // In IE 6/7, searching for 'msg' turns up the 'html:msg', but searching for 'html:msg' does not.
 43+ // In later IE and other browsers, searching for 'html:msg' turns up the 'html:msg', but searching for 'msg' does not.
 44+ // So searching for both 'msg' and 'html:msg' seems to get the job done.
 45+ // This feels pretty icky, though.
 46+ .find( 'msg,html\\:msg' )
4247 .each( function() {
4348 var $el = $(this);
4449 $el
Property changes on: branches/REL1_18/phase3
___________________________________________________________________
Modified: svn:mergeinfo
4550 Merged /trunk/phase3:r92758

Past revisions this follows-up on

RevisionCommit summaryAuthorDate
r92758Followup r92329: Fixes the localize test cases in IE 6 and 7....brion17:45, 21 July 2011

Status & tagging log