User "Krinkle" posted a comment on MediaWiki.r97150.

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

Update jquery.tablesorter for r97145: emulate <thead> if there is no <thead> in 
the HTML, by walking down the table starting at the first row and moving rows 
to the <thead> as long as all of its cells are <th>s (or the row is empty). 
Also fix and simplify the sortbottom code, which was incorrectly creating 
multiple <tfoot> elements if there were multiple sortbottom rows.

Comment:

Okay, I guess it depends on what the criteria are. Not allowing further rows to 
be added makes sense, yes. (since that would change the visual appearance). 
Either way, it still needs a unit test to verify that it works.

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

Reply via email to