[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