Tablesort outputs invalid XHTML by adding a 'field' attribute to <th>s.

CommentFileSizeAuthor
#3 th_attribute.patch390 bytesAnonymous (not verified)
#1 th_field_attribute.patch369 bytesAnonymous (not verified)
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Anonymous’s picture

FileSize
369 bytes

Patch to fix the issue in includes/common.inc...

Dries’s picture

Assigned: Unassigned » Dries

Commited to the DRUPAL-4-3 branch and ported to HEAD. Thanks.

Anonymous’s picture

FileSize
390 bytes

Sorry, patch didn't fix whole problem (Tablesort is also outputting a 'sort' attribute to <th>s). Here's a patch which actually removes both the 'field' and 'sort' attribute and thus produces valid XHTML. The patch is for tablesort.inc rather than common.inc as this seemed a far more sensible place to make the fix.

Dries’s picture

Agreed. Undid the previous changes and commited your patch to the DRUPAL-4-3 branch. Changes ported to HEAD as well. Thanks.

Anonymous’s picture