[MediaWiki-CodeReview] [MediaWiki r81583]: New comment added

2011-08-05 Thread MediaWiki Mail
User ^demon posted a comment on MediaWiki.r81583.

Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/81583#c20488
Commit summary:

Switch editsection to mw:editsection and start hiding these editsection 
tags from Tidy so it doesn't break them.

Comment:

* Definitely needs some unit tests
* If we're going to have a class for Tidy-related stuff, it might make sense to 
move more of Parser's tidy code into it

___
MediaWiki-CodeReview mailing list
mediawiki-coderev...@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-codereview


[MediaWiki-CodeReview] [MediaWiki r81583]: New comment added

2011-07-15 Thread MediaWiki Mail
User Dantman posted a comment on MediaWiki.r81583.

Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/81583#c19683
Commit summary:

Switch editsection to mw:editsection and start hiding these editsection 
tags from Tidy so it doesn't break them.

Comment:

Whoops, the replace code used to be part of the Tidy class until I decided it 
would be better to split it out. Feel free to cut that out.

___
MediaWiki-CodeReview mailing list
mediawiki-coderev...@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-codereview


[MediaWiki-CodeReview] [MediaWiki r81583]: New comment added

2011-07-13 Thread MediaWiki Mail
User Aaron Schulz posted a comment on MediaWiki.r81583.

Full URL: http://www.mediawiki.org/wiki/Special:Code/MediaWiki/81583#c19570
Commit summary:

Switch editsection to mw:editsection and start hiding these editsection 
tags from Tidy so it doesn't break them.

Comment:

What is the empty replaceEditSectionLinksCallback() there for?

___
MediaWiki-CodeReview mailing list
mediawiki-coderev...@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-codereview