Answers

• Loren Maxwell Posts: 387Questions: 94Answers: 10

  May 2020 edited May 2020

  For convenience, I'm adding a case from the DataTables site:
  http://live.datatables.net/pomeyuzo/1/edit

  To duplicate:
  1) Initially the table is fine
  2) Change the JS from "orderSequence": [ "asc", "desc" ] to "orderSequence": [ "desc", "desc" ] or "orderSequence": [ "asc", "asc" ] to disable sorting in some direction
  3) Sort by the Age column and then sort by any other column and notice that when the class sorting_desc_disabled or sorting_asc_disabled is applied to the Age column, the column header is no longer aligned
• colin Posts: 15,179Questions: 1Answers: 2,590

  May 2020

  Thanks for the test case - I'm not sure if I'm missing something, but it seems to work for me - see video here. Please could you take a look and see if I'm doing something wrong.

  Colin
• Loren Maxwell Posts: 387Questions: 94Answers: 10

  May 2020

  Thanks, @collin,

  The video works the same as the test case, but I'm surprised the header text should change position.

  Perhaps a better demonstration of what I'm referring to can be seen on my page (it takes a couple of seconds to load):
  https://ghsfha.org/w/Special:SchoolIndex

  Notice how initially the sort icon is on sitting on top of the "Loss", "Tie", etc., column text.

  When you click on "Loss", the text shifts into what I'd consider the proper place but sorting_asc_disabled is now applied to the "Win" column and now that icon is sitting on top of the "Win" text.

  Also, it seems like when using Scroller, this spills over into the column header width so that they don't match the table with the actual data, but I can't find an example offhand . . .
• allan Posts: 61,972Questions: 1Answers: 10,160 Site admin

  May 2020 Answer ✓

  Yup - looks like a bug there I'm afraid. I've just committed a fix and it will be in the nightly shortly.

  Regards,
  Allan
• Loren Maxwell Posts: 387Questions: 94Answers: 10

  May 2020

  @allan -- There's still something slightly off when using Scroller and the sorting_disabled class applied.

  Here's a test case from my webpage, although note that you might have view this page with a narrow browser window to see what I'm referring to:

  https://ghsfha.org/wiki/index.php?title=Special:DatabaseHome&view=coach

  When using Scollerm, the sorting_disabled class causes the column headers to be misaligned because the columns where it is applied are too short.

  When I manually remove the sorting_disabled class from that column header everything lines back up.

  In the css itself, when I remove these references, everything lines up:

              &:not(.sorting_disabled) {
                  padding-right: 30px;
              }
  
• allan Posts: 61,972Questions: 1Answers: 10,160 Site admin

  June 2020

  I think this might actually be a problem with the use of multiple header rows - for example the first cell has 30px padding right on the top row, but not on the second row.

  I'm not immediately sure how to handle that in DataTables, other than to add 30px padding on every cell regardless. Alternatively, you could manually add a suitable class to add the 30px padding and less (looks like 20px) to make sure the padding is the same over the vertical alignment of cells. But the wheels will come off that with colspan, hence I'm thinking 30px always might be required...

  Allan
• Loren Maxwell Posts: 387Questions: 94Answers: 10

  June 2020 edited June 2020

  Thanks, @allan, I didn't even pay attention to the top row at all when troubleshooting . . .

  When you say:

  "I'm not immediately sure how to handle that in DataTables, other than to add 30px padding on every cell regardless . . . I'm thinking 30px always might be required..."

  Are you considering changing Datatables to include that? Or was that a suggestion for me to do on my tables?

  I can do that for my tables, but I can also wait if there's a broader solution you were thinking of implementing.
• allan Posts: 61,972Questions: 1Answers: 10,160 Site admin

  June 2020

  I'm not sure yet to be honest! It would be nice to avoid doing it if possible, but I currently don't see how we can. I'm going to have to think on it a bit.

  Allan