Closed (won't fix)
Project:
Case Tracker
Version:
6.x-1.x-dev
Component:
Code
Priority:
Minor
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
21 Jan 2010 at 15:42 UTC
Updated:
4 Oct 2013 at 06:27 UTC
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
Comment #1
jmiccolis commentedNonsense! 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.
Comment #2
pedrorocha commentedI'm cleaning the module issues, so if somebody thinks that this issue does make sense yet, feel free to reopen.
Comment #3
joachim commentedDepends, does casetracker still have this problem on D7?