Tables in the case form are semantically incorrect.

The assignment table also causes the need for fiddled in the theme function for #464834: Permission to assign case to account for access.

Use floats instead maybe? Perhaps either code or CSS from theme_links?

    $header = array_fill(0, 5, array());
    $header[0] = $form['assign_to']['#title'];
    $radios = array();
    foreach (element_children($form['assign_to']) as $id) {
      $radios[] = drupal_render($form['assign_to'][$id]);
    }
    $radios = array_chunk($radios, 5);
    $output .= theme('table', $header, $radios, array('class' => 'casetracker-assign-to'));
    drupal_render($form['assign_to']);

Comments

jmiccolis’s picture

Priority: Normal » Minor

Nonsense! Drupal has a grand history of using tables in forms: see admin/build/modules, etc...

:P

Seriously however, I'm willing to look at patches here but not inclined to spend time on this myself.

pedrorocha’s picture

Status: Active » Closed (won't fix)

I'm cleaning the module issues, so if somebody thinks that this issue does make sense yet, feel free to reopen.

joachim’s picture

Depends, does casetracker still have this problem on D7?