Right above the "Parameters" section is the following:

Here's an example for $rows: @verbatim $rows = array( // Simple row array( 'Cell 1', 'Cell 2', 'Cell 3' ), // Row with attributes on the row and some of its cells. array( 'data' => array('Cell 1', array('data' => 'Cell 2', 'colspan' => 2)), 'class' => 'funky' ) ); @endverbatim

Presumably this is meant to display as, well, verbatim - kind of hard to read as just text.

Thanks.

CommentFileSizeAuthor
#3 669374-D6.patch2.19 KBjhodgdon
#2 669374.patch2.24 KBjhodgdon
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jhodgdon’s picture

Title: Documentation problem with theme_table » theme_table doc uses @verbatim instead of @code
Version: 6.x-dev » 7.x-dev
Issue tags: +Novice

Good catch! This should be using @code instead of @verbatim.

It's also a problem in the D7 version of this doc, so should be fixed there and then backported to D6.
http://api.drupal.org/api/function/theme_table/7

jhodgdon’s picture

Status: Active » Needs review
FileSize
2.24 KB

Here's a patch for Drupal 7. I found one other place in the D7 source using @verbatim.

jhodgdon’s picture

Status: Needs review » Reviewed & tested by the community
FileSize
2.19 KB

Here's a Drupal 6 patch. I don't think either of these is in the least bit controversial, so I've set status to RTBC, hope that's OK. If the D7 patch is committed, please change version to D6 so this can be committed there too.

webchick’s picture

Version: 7.x-dev » 6.x-dev

Committed to HEAD. Marking down to 6.x.

Dries’s picture

Committed to CVS HEAD. Moving to D6.

jhodgdon’s picture

echo. Wonder which one of you committed this? :)

Gábor Hojtsy’s picture

Status: Reviewed & tested by the community » Fixed

Thanks, committed to Drupal 6 as well.

Status: Fixed » Closed (fixed)
Issue tags: -Novice

Automatically closed -- issue fixed for 2 weeks with no activity.