Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
#2645492: Accessibility Patch - Adding Scope Attribute not only added scope
attributes, but also changed the (usually empty) first header cell to use an <td>
element instead of <th>
. While this does makes sense in some way, of course, since an empty cell isn't a header for anything, it breaks Drupal Core's (notoriously unflexible) "Sticky table headers" functionality. (See, e.g., here – <td>
elements are just not considered in some parts of the code as potentially being part of the header.)
Comment | File | Size | Author |
---|---|---|---|
#2 | 2893748-2--keep_header_th_elements.patch | 1.52 KB | drunken monkey |
Comments
Comment #2
drunken monkeyThe attached patch would revert that part of the patch. It correctly fixes the sticky table headers for me.
Comment #4
drunken monkeyCommitted.