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

Kobus drupal-devel at drupal.org
Mon Sep 5 08:22:55 UTC 2005


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:   Kobus
 Status:       patch (code needs review)

Yes, indeed, you are correct. Pardon my oversight that it is already a
attribute. I evaluated homepage reader about 2 months ago, and merely
recalled summary. Sorry!


Kobus




Kobus



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.




------------------------------------------------------------------------

Mon, 05 Sep 2005 08:20:54 +0000 : Bèr Kessels

+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.







More information about the drupal-devel mailing list