| Esanders | |
| Mar 20 2019, 1:37 PM |
| F28446090: image.png | |
| Mar 22 2019, 8:25 PM |
| F28446083: image.png | |
| Mar 22 2019, 8:25 PM |
| F28425330: image.png | |
| Mar 20 2019, 1:54 PM |
| F28425340: image.png | |
| Mar 20 2019, 1:54 PM |
Our previous attempt to fix this was backed out when it was discovered the CSS selector did not match with what the JS did, so it broke complex tables (T195108).
We can still fix this for the majority of simpler tables, and avoid breaking complex tables by simply removing the CSS approximation once the JS has run.
Change 497769 had a related patch set uploaded (by Esanders; owner: Esanders):
[mediawiki/core@master] TableSorter: Fix pre-JS selector to match wikitext-generated sortable tables
After the above patch here is the change in appearance for a simple sortable table, and @matmarex 's minimal failing test case from the previous attempt:
| before | after |
|---|---|
 |  |
Well it now fails for tables with row-header cells…
{| class="wikitable sortable"
!class="unsortable"|
!A
!B
|-
!α
|4
|3
|-
!β
|2
|1
|}| before JS | after JS |
|---|---|
 |  |
I don't think there is a viable CSS solution for this problem. It can only be solved by implementing the tablesorter header cells algorithm in the PHP parser.
I think a solution that results in FOUC in <10% of cases is better than one that results in a FOUC in 100% of cases.
Amended the selector to be limited to the first row. This will remove the icon from the H1/H2 second row headers in the first example, causing a width change, but fixes the row-headers case which is probably more common.
Change 497769 merged by jenkins-bot:
[mediawiki/core@master] jquery.tablesorter: Fix pre-JS selector to match wikitext-generated sortable tables