[drupal-devel] Move pagers out of table
Dries Buytaert
dries at buytaert.net
Sat Sep 24 08:06:27 UTC 2005
Hello world,
I just committed a Neil Drumm production: "move pagers out of
table". See http://drupal.org/node/28786.
Moving the pagers out of the tables isn't strictly necessary but it
makes for better XHTML and less code (for core: 31 insertions and 66
deletions).
If you maintain a module in the contributions repository, you might
want to update it for sake of consistency. A typical change looks
like this:
- if ($pager = theme('pager', NULL, 50, 0)) {
- $rows[] = array(array('data' => $pager, 'colspan' => '7'));
- }
$output .= theme('table', $header, $rows);
+ $output .= theme('pager', NULL, 50, 0);
The following modules could be updated for consistency with core:
actions/actions.module
akvaforum/akvaforum.module
album/album.module
asterisk/asterisk.module
automail/am.module
badbehavior/badbehavior.module
banner/banner.module
bookmarks/bookmarks.module
bookreview/bookreview.module
browscap/browscap.module
checkout/checkout.module
contact_dir/contact_dir.module
dba/dba.module
distantparent/distantparent.module
eatlocal/foodmiles/foodmiles.module
eatlocal/resource/resource.module
ecommerce/contrib/submanage/submanage.module
ecommerce/product/product.module
ecommerce/store/store.module
epublish/epublish.module
evaluation/evaluation.module
go/go.module
groups/groups.module
jobsearch/patches/node.module-4.6.2
jobsearch/patches/node.module-4.6.3
jobsearch/patches/tracker.module-4.6.2
links/links.module
logsearch/logsearch.module
mail_archive/mail_archive.module
moviereview/moviereview.module
netnews/netnews.module
nmoderation/nmoderation.module
og/og.module
paypal_framework/paypal_framework.module
paypal_subscription/paypal_subscription.module
privatemsg/privatemsg.module
project/issue.inc
queue/queue.module
scheduler/scheduler.module
scouting/person.inc
simplenews/simplenews.module
spam/optional/comment.module.patch
spam/spam.module
survey/survey.module
task/task.module
taxonomy_assoc/taxonomy_assoc.module
term_statistics/term_statistics.module
trackback/trackback.module
translation/translation.module
troll/troll.module
userpoints/userpoints.module
volunteer/volunteer.module
wireframe/wireframe.module
workspace/workspace.module
Thanks,
--
Dries Buytaert :: http://www.buytaert.net/
More information about the drupal-devel
mailing list