diff options
author | Dries Buytaert <dries@buytaert.net> | 2005-03-05 09:09:07 +0000 |
---|---|---|
committer | Dries Buytaert <dries@buytaert.net> | 2005-03-05 09:09:07 +0000 |
commit | 11dd94bde5e9417e81b8ce4dd4fc2d8710dbdd82 (patch) | |
tree | 35149e66f4180a1d46d7f44743e8ef8ef060a274 /modules/block.module | |
parent | 5c4fb7a84519a5a8e264862115a705e13bb23d3e (diff) | |
download | brdo-11dd94bde5e9417e81b8ce4dd4fc2d8710dbdd82.tar.gz brdo-11dd94bde5e9417e81b8ce4dd4fc2d8710dbdd82.tar.bz2 |
- Patch #18382 by Neil: usability improvement: organized the blocks on the block administration page by region. (I wanted to do that myself!)
Diffstat (limited to 'modules/block.module')
-rw-r--r-- | modules/block.module | 39 |
1 files changed, 29 insertions, 10 deletions
diff --git a/modules/block.module b/modules/block.module index 359a78e6c..733e9a3e4 100644 --- a/modules/block.module +++ b/modules/block.module @@ -189,7 +189,9 @@ function block_admin_display() { } $header[] = array('data' => t('Operations'), 'colspan' => 2); - + $left = array(); + $right = array(); + $disabled = array(); foreach ($blocks as $block) { if ($block['module'] == 'block') { $delete = l(t('delete'), 'admin/block/delete/'. $block['delta']); @@ -198,13 +200,7 @@ function block_admin_display() { $delete = ''; } - $row = array($block['info'], - form_checkbox(NULL, $block['module'] .']['. $block['delta'] .'][status', 1, $block['status']), - form_weight(NULL, $block['module'] .']['. $block['delta'] .'][weight', $block['weight']), - form_radios(NULL, $block['module'] .']['. $block['delta'] .'][region', $block['region'], - array(t('left'), t('right')))); - - $row = array($block['info'], + $row = array(array('data' => $block['info'], 'class' => 'block'), form_checkbox(NULL, $block['module'] .']['. $block['delta'] .'][status', 1, $block['status']), form_weight(NULL, $block['module'] .']['. $block['delta'] .'][weight', $block['weight']), form_radios(NULL, $block['module'] .']['. $block['delta'] .'][region', $block['region'], @@ -215,10 +211,33 @@ function block_admin_display() { } $row[] = l(t('configure'), 'admin/block/configure/'. $block['module'] .'/'. $block['delta']); $row[] = $delete; - $rows[] = $row; + if ($block['status']) { + if ($block['region'] == 0) { + $left[] = $row; + } + else { + $right[] = $row; + } + } + else { + $disabled[] = $row; + } } - $output = theme('table', $header, $rows); + $rows = array(); + if (count($left)) { + $rows[] = array(array('data' => t('Left sidebar'), 'class' => 'region', 'colspan' => (module_exist('throttle') ? 7 : 6))); + $rows = array_merge($rows, $left); + } + if (count($right)) { + $rows[] = array(array('data' => t('Right sidabar'), 'class' => 'region', 'colspan' => (module_exist('throttle') ? 7 : 6))); + $rows = array_merge($rows, $right); + } + if (count($disabled)) { + $rows[] = array(array('data' => t('Disabled'), 'class' => 'region', 'colspan' => (module_exist('throttle') ? 7 : 6))); + $rows = array_merge($rows, $disabled); + } + $output = theme('table', $header, $rows, array('id' => 'blocks')); $output .= form_submit(t('Save blocks')); return form($output, 'post', url('admin/block')); |