#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.)

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

drunken monkey created an issue. See original summary.

drunken monkey’s picture

Status: Active » Needs review
FileSize
1.52 KB

The attached patch would revert that part of the patch. It correctly fixes the sticky table headers for me.

  • drunken monkey committed b61aaff on 7.x-1.x
    Issue #2893748 by drunken monkey: Fixed sticky table header...
drunken monkey’s picture

Status: Needs review » Fixed

Committed.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.