When creating tables using the theme_table function, the default method to turn off table striping is to add the no_striping option row by row:
$rows[] = array('data' => array(...), 'no_striping' => TRUE);

When using Bootstrap, however, the table-striped class is always added to the table. I would like to propose that users add the no_striping class to a table when striping is not desired (since it usually is, I think making striping by default is good):
theme('table', array('header' => $header, 'rows' => $rows, 'attributes' => array('class' => 'no_striping')));

Please see attached patch, where bootstrap_preprocess_table is modified to look for this class before adding the table-striped class.

CommentFileSizeAuthor
#1 Add_option_to_turn_table-striped_class_off-1911218-1.patch460 bytesMD3
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

MD3’s picture

MD3 CreditAttribution: MD3 commented
FileSize
Add_option_to_turn_table-striped_class_off-1911218-1.patch460 bytes

Patch.

wundo’s picture

wundo CreditAttribution: wundo commented
Status: Needs review » Fixed

Commited! :)

But I'm using the following class to toggle no stripping: table-no-striping

MD3’s picture

MD3 CreditAttribution: MD3 commented

Awesome! Thanks so much!

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

sobi3ch’s picture

sobi3ch CreditAttribution: sobi3ch commented
Issue summary: View changes

There is now other class to do it table-no-striping. So just theme table like this:

  return theme('table',  array(
    'header' => $headers, 
    'rows'=> $rows,
    'attributes' => array('class' => 'table-no-striping')
  ));