Use of expand-child class to define subordinate/child table row breaks table sorting function. Example at https://en.wikipedia.org/wiki/2014_term_United_States_Supreme_Court_opinions_of_Ruth_Bader_Ginsburg. Loss of function occurred relatively recently, could be caused by change to colspan--see comment at https://en.wikipedia.org/w/index.php?title=Wikipedia:Village_pump_%28technical%29&diff=684281564&oldid=684271506 .
Description
Description
Details
Details
| Subject | Repo | Branch | Lines +/- | |
|---|---|---|---|---|
| jquery.tablesorter: use of expand-child class | mediawiki/core | master | +39 -2 |
Related Objects
Related Objects
Event Timeline
Comment Actions
In manageColspans(), cssChildRows are skipped and don't receive columnToCell data property. However in detectParserForColumn() the data in columnToCell is used unconditionally.
Comment Actions
Change 244940 had a related patch set uploaded (by MatthiasDD):
jquery.tablesorter: use of expand-child class
Comment Actions
In the past cssChildRows are read for auto detection of parser only for the first row. Now cssChildRows are scipped for auto detection and sorting.
After change I5180296 i have tested the example in https://en.wikipedia.org/wiki/Help:Sorting#Keeping some rows together. But this example have only text in the first row and in this case the second row was not read.