[drupal-devel] [feature] Add <caption> support to theme_table()

Kobus drupal-devel at drupal.org
Mon Sep 5 07:19:14 UTC 2005

Issue status update for 
Post a follow up: 

 Project:      Drupal
 Version:      cvs
 Component:    theme system
 Category:     feature requests
 Priority:     normal
 Assigned to:  Anonymous
 Reported by:  Crell
 Updated by:   Kobus
 Status:       patch (code needs review)


I won't comment on the implementation, because I don't know the impact
of this, but from a usability pespective it is a HUGE +1 for me, as
some screen readers use either the  or  tag to describe table contents
to blind users. An example of a browser/screen reader integration that
uses the   tag is IBM's Homepage Reader.




Previous comments:

Mon, 05 Sep 2005 07:12:18 +0000 : Crell

Attachment: http://drupal.org/files/issues/theme_table_0.patch (1001 bytes)

Well, if I made this patch properly :-), this patch adds a fourth
parameter to the theme_table() function, $caption.  If set, it adds a
<caption>$caption</caption> line to the table.  If it is not set, this
patch has no effect.  All styling is left up to CSS to figure out.  

It seems to me that $caption would make more sense as the 3rd
parameter, before $attributes, but that would break backward
compatibility so I just tacked it on at the end.  I'll leave the final
order of parameters up to someone else to figure out.

More information about the drupal-devel mailing list