Page MenuHomePhabricator

Table sorting broken by use of expand-child class
Closed, ResolvedPublic


Use of expand-child class to define subordinate/child table row breaks table sorting function. Example at Loss of function occurred relatively recently, could be caused by change to colspan--see comment at .


Related Gerrit Patches:

Event Timeline

Postdlf created this task.Oct 5 2015, 11:20 PM
Postdlf raised the priority of this task from to Needs Triage.
Postdlf updated the task description. (Show Details)
Postdlf added a subscriber: Postdlf.
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptOct 5 2015, 11:20 PM
TheDJ added a subscriber: TheDJ.EditedOct 6 2015, 11:54 AM

Broken by the change that fixed T105731 and T74534

In manageColspans(), cssChildRows are skipped and don't receive columnToCell data property. However in detectParserForColumn() the data in columnToCell is used unconditionally.

Change 244940 had a related patch set uploaded (by MatthiasDD):
jquery.tablesorter: use of expand-child class

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 some rows together. But this example have only text in the first row and in this case the second row was not read.

TheDJ closed this task as Resolved.Oct 13 2015, 7:46 PM
TheDJ assigned this task to MatthiasDD.
TheDJ triaged this task as Medium priority.
TheDJ removed a project: Patch-For-Review.
TheDJ set Security to None.

Change 244940 merged by jenkins-bot:
jquery.tablesorter: use of expand-child class