Issue status update for http://drupal.org/node/30423 Post a follow up: http://drupal.org/project/comments/add/30423 Project: Drupal Version: cvs Component: theme system Category: feature requests Priority: normal Assigned to: Anonymous Reported by: Crell Updated by: Bèr Kessels Status: patch (code needs review) +1 from me too. This is a huge accessability improvement, yet a very small patch. We should follow this patch up, with a list of theme('table') calls that should use this caption. Bèr Kessels 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. ------------------------------------------------------------------------ Mon, 05 Sep 2005 07:19:10 +0000 : Kobus Hi, 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. Regards, Kobus ------------------------------------------------------------------------ Mon, 05 Sep 2005 07:20:23 +0000 : Kobus Hi, 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 <summary> or <caption> tag to describe table contents to blind users. An example of a browser/screen reader integration that uses the <summary> tag is IBM's Homepage Reader. Regards, Kobus ------------------------------------------------------------------------ Mon, 05 Sep 2005 07:45:29 +0000 : Crell Well, summary is an attribute of <table>, not an element in itself. You can already set that with the $attributes parameter. I don't know if the Powers That Be would want another parameter just for that attribute, although that might increase its usage. I just needed the caption because I have two tables on one admin page, and not having some way to tell them apart was not good.