Use thead and tbody in theme_table()

+1 on the direction. But we also need to add TFOOT.

All themed tables should have the ability to add THEAD,TFOOT, and TBODY to support true semantic table building.

Committed to HEAD. Thanks.

Maybe look into http://drupal.org/node/28777 next?

Never mind my previous comment. Attached it to the wrong issue.

I would actually say NO to tfoot, on the grounds that the HTML spec is broken with regards to tfoot. It doesn't degrade gracefully. An unrecognized thead has essentially no effect. An unrecognized tfoot means that the footer row gets rendered above the tbody, but if recognized then it appears after the tbody. I don't know what W3C moron came up with that idea, but that's the reason why I never use tfoot. :-)

Besides, there's no current use of what would go into a footer in Drupal, as far as I know. We do, however, already have a $header parameter, so wrapping that in thead makes sense. The patch will need to be updated, though, since http://drupal.org/node/30423 updates this function as well.

I am curious why the switch from $attributes = NULL to $attribures = array(), though. What's the reasoning for that? (I'm not opposed necessarily, just curious.)

Patch no longer applies.

Is this still open?

Patch applies cleanly to HEAD, doesn't break anything, and obeys coding conventions. Let's get this one committed.

Committed to HEAD.