We now always do explodeRowspans before the first sort, to accomodate the new .sort() function. This has the unintended consequence that you basically cannot use rowspans at all anymore, since the JS code will always immediately fragment them back into individual cells right now.
This is a side effect of: https://gerrit.wikimedia.org/r/22562
Version: 1.21.x
Severity: critical
If I'm understanding this right, this has essentially broken rowspan? Given the sheer number of pages this has altered I'm upping the priority - I'll try to give the bugmeister a kick.
Krinkle, Sam: Can you take a look at this, please?
High priority: sortable && rowspans don't work together at all, pretty visible for readers.
Example with "sortable" still enabled: https://en.wikipedia.org/w/index.php?title=2012_in_film&oldid=521988570#January.E2.80.93March
Potentially related past code changes (quick check):
https://gerrit.wikimedia.org/r/#/c/30649/ Update jQueryUI to 1.9.1
https://gerrit.wikimedia.org/r/#/c/23100/ make $.tablesorter treat alt like text
Copying from the thread on https://en.wikipedia.org/wiki/Wikipedia:Village_pump_%28technical%29#Javascript_screwing_up_tables :
"It appears that clicking the nth column header actually did sort the nth column, but the first header has colspan="3". For all subsequent headers this means that the column being sorted is two columns to the left of the clicked header. Any chance the table sorter could be modified to handle this?" PrimeHunter 14:14, 8 November 2012 (UTC)
Note to myself: The previous issue for this example when values were shown in wrong table cells (making the table completely unreadable) was a markup issue: Using ! instead of | in this specific table: https://en.wikipedia.org/w/index.php?title=2012_in_film&diff=521988570&oldid=521959220
sumanah wrote:
I agree that this is not HIGHEST priority, because there's no data loss, but it's pretty high. CC'ing Hoo & Matmarex to see whether they can provide a quick JS fix.
If lots of tables across the wikis are mistakenly using ! where they should have used |, and MediaWiki used to be forgiving and now it's not and the tables croak, we should address that (either with an educational campaign, a regex, or building some kind of forgiveness into the rendering).
Yeah. If (as I understand it) the VE will standardise formatting on save, it might be something worth temporarily punting on and asking them to sort out if/when they deploy.
(In reply to comment #4)
But that last item (! instead of |) would be a separate bug.
-> bug 41889
Changing description from "tablesorter explodes rowspans now before ever
clicking sort" to "tablesorter doesn't handle headers with colspan properly",
as I understand that this is the real problem.
I'll try to look into it.
Gerrit changeset 32593 fixes the original bug reported here (rowspans being exploded automatically).
Actually, not fixed yet; sorry for the confusion.
This bug would be fixed by change I89766c96 (mine), which isn't yet merged; the merged one, Ibe4cc7e9 (Brad's), fixes bug 41889 (which was split from this one).
(In reply to comment #12)
Actually, not fixed yet; sorry for the confusion.
This bug would be fixed by change I89766c96 (mine), which isn't yet merged; the
merged one, Ibe4cc7e9 (Brad's), fixes bug 41889 (which was split from this
one).
There are actually three issues mentioned here:
Brad: Thanks for the analysis!
The patches for 1. and 2. are merged, so only Icb674f7e for bug 41889 is left.
Since 1 and 2 are both merged now, I'm going to close this bug. Bug 41889 is still open for that issue.