User "Hashar" changed the status of MediaWiki.r94681.

Old Status: new
New Status: ok

Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/94681
Commit summary:

Followup r94609: fix jquery.highlightText test cases on IE, simplify test case 
setup so new ones can be added more trivially, uncommented the failing test 
case.

These tests were failing on IE 6, 7, and 8 because the innerHTML rendering of 
the highlighted text came out in caps and without quotes, such that the string 
didn't match the provided one. By re-normalizing the expected HTML form this 
check now works on those browsers. Switching common code to a loop and an array 
makes it easier to add new test cases: less duplication of code, and no need to 
manually update the number of test cases.

Note that the third test case was previously commented out, but looks like a 
serious regression. No longer commented out, so this will fail!

_______________________________________________
MediaWiki-CodeReview mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-codereview

Reply via email to