diff -urNp /Users/kris/drupalhead/drupal/includes/pager.inc /Users/kris/drupalhead/drupal patched/includes/pager.inc
--- /Users/kris/drupalhead/drupal/includes/pager.inc 2007-02-15 05:40:17.000000000 -0600
+++ /Users/kris/drupalhead/drupal patched/includes/pager.inc 2007-04-15 17:06:05.000000000 -0500
@@ -95,6 +95,7 @@ function pager_get_querystring() {
*
* Menu callbacks that display paged query results should call theme('pager') to
* retrieve a pager control so that users can view other results.
+ * Format a list of nearby pages with additional query results.
*
* @param $tags
* An array of labels for the controls in the pager.
@@ -104,28 +105,104 @@ function pager_get_querystring() {
* An optional integer to distinguish between multiple pagers on one page.
* @param $parameters
* An associative array of query string parameters to append to the pager links.
+ * @param $quantity
+ * The number of pages in the list.
* @return
* An HTML string that generates the query pager.
*
* @ingroup themeable
*/
-function theme_pager($tags = array(), $limit = 10, $element = 0, $parameters = array()) {
- global $pager_total;
- $output = '';
+function theme_pager($tags = array(), $limit = 10, $element = 0, $parameters = array(), $quantity = 5) {
+ global $pager_page_array, $pager_total;
- if ($pager_total[$element] > 1) {
- $output .= '';
- return $output;
+ // Calculate various markers within this pager piece:
+ // Middle is used to "center" pages around the current page.
+ $pager_middle = ceil($quantity / 2);
+ // current is the page we are currently paged to
+ $pager_current = $pager_page_array[$element] + 1;
+ // first is the first page listed by this pager piece (re quantity)
+ $pager_first = $pager_current - $pager_middle + 1;
+ // last is the last page listed by this pager piece (re quantity)
+ $pager_last = $pager_current + $quantity - $pager_middle;
+ // max is the maximum page number
+ $pager_max = $pager_total[$element];
+ // End of marker calculations.
+
+ // Prepare for generation loop.
+ $i = $pager_first;
+ if ($pager_last > $pager_max) {
+ // Adjust "center" if at end of query.
+ $i = $i + ($pager_max - $pager_last);
+ $pager_last = $pager_max;
+ }
+ if ($i <= 0) {
+ // Adjust "center" if at start of query.
+ $pager_last = $pager_last + (1 - $i);
+ $i = 1;
+ }
+ // End of generation loop preparation.
+
+ if ($pager_total[$element] > 1) {
+ $items[] = array(
+ 'class' => 'pager-first',
+ 'data' => theme('pager_first', ($tags[0] ? $tags[0] : t('« first')), $limit, $element, $parameters),
+ );
+ $items[] = array(
+ 'class' => 'pager-previous',
+ 'data' => theme('pager_previous', ($tags[1] ? $tags[1] : t('‹ previous')), $limit, $element, 1, $parameters),
+ );
+
+ // When there is more than one page, create the pager list.
+ if ($i != $pager_max) {
+ if ($i > 1) {
+ $items[] = array(
+ 'class' => 'pager-ellipsis',
+ 'data' => '…',
+ );
+ }
+ // Now generate the actual pager piece.
+ for (; $i <= $pager_last && $i <= $pager_max; $i++) {
+ if ($i < $pager_current) {
+ $items[] = array(
+ 'class' => 'pager-item',
+ 'data' => theme('pager_previous', $i, $limit, $element, ($pager_current - $i), $parameters),
+ );
+ }
+ if ($i == $pager_current) {
+ $items[] = array(
+ 'class' => 'pager-current',
+ 'data' => $i,
+ );
+ }
+ if ($i > $pager_current) {
+ $items[] = array(
+ 'class' => 'pager-item',
+ 'data' => theme('pager_next', $i, $limit, $element, ($i - $pager_current), $parameters),
+ );
+ }
+ }
+ if ($i < $pager_max) {
+ $items[] = array(
+ 'class' => 'pager-ellipsis',
+ 'data' => '…',
+ );
+ }
+ }
+ // End generation.
+ $items[] = array(
+ 'class' => 'pager-next',
+ 'data' => theme('pager_next', ($tags[3] ? $tags[3] : t('next ›')), $limit, $element, 1, $parameters),
+ );
+ $items[] = array(
+ 'class' => 'pager-last',
+ 'data' => theme('pager_last', ($tags[4] ? $tags[4] : t('last »')), $limit, $element, $parameters),
+ );
+ return theme('item_list', $items, NULL, 'ul', array('class' => 'pager'));
}
}
+
/**
* @name Pager pieces
* @{
@@ -155,7 +232,11 @@ function theme_pager_first($text, $limit
// If we are anywhere but the first page
if ($pager_page_array[$element] > 0) {
- $output = theme('pager_link', $text, pager_load_array(0, $element, $pager_page_array), $element, $parameters, array('class' => 'pager-first'));
+ $output = theme('pager_link', $text, pager_load_array(0, $element, $pager_page_array), $element, $parameters);
+ }
+ // Print text for positioning purposes without a link
+ else {
+ $output = $text;
}
return $output;
@@ -193,10 +274,14 @@ function theme_pager_previous($text, $li
}
// The previous page is not the first page.
else {
- $output = theme('pager_link', $text, $page_new, $element, $parameters, array('class' => 'pager-previous'));
+ $output = theme('pager_link', $text, $page_new, $element, $parameters);
}
}
-
+ // Print text for positioning purposes without a link
+ else {
+ $output = $text;
+ }
+
return $output;
}
@@ -231,9 +316,13 @@ function theme_pager_next($text, $limit,
}
// The next page is not the last page.
else {
- $output = theme('pager_link', $text, $page_new, $element, $parameters, array('class' => 'pager-next'));
+ $output = theme('pager_link', $text, $page_new, $element, $parameters);
}
}
+ // Print text for positioning purposes without a link
+ else {
+ $output = $text;
+ }
return $output;
}
@@ -260,90 +349,17 @@ function theme_pager_last($text, $limit,
// If we are anywhere but the last page
if ($pager_page_array[$element] < ($pager_total[$element] - 1)) {
- $output = theme('pager_link', $text, pager_load_array($pager_total[$element] - 1, $element, $pager_page_array), $element, $parameters, array('class' => 'pager-last'));
+ $output = theme('pager_link', $text, pager_load_array($pager_total[$element] - 1, $element, $pager_page_array), $element, $parameters);
}
-
- return $output;
-}
-
-/**
- * Format a list of nearby pages with additional query results.
- *
- * @param $limit
- * The number of query results to display per page.
- * @param $element
- * An optional integer to distinguish between multiple pagers on one page.
- * @param $quantity
- * The number of pages in the list.
- * @param $text
- * A string of text to display before the page list.
- * @param $parameters
- * An associative array of query string parameters to append to the pager links.
- * @return
- * An HTML string that generates this piece of the query pager.
- *
- * @ingroup themeable
- */
-function theme_pager_list($limit, $element = 0, $quantity = 5, $text = '', $parameters = array()) {
- global $pager_page_array, $pager_total;
-
- $output = '';
return $output;
}
+
/**
* Format a link to a specific query result page.
*
@@ -417,6 +433,4 @@ function pager_load_array($value, $eleme
// Update the changed element.
$new_array[$element] = (int)$value;
return $new_array;
-}
-
-
+}
\ No newline at end of file
diff -urNp /Users/kris/drupalhead/drupal/modules/system/system.css /Users/kris/drupalhead/drupal patched/modules/system/system.css
--- /Users/kris/drupalhead/drupal/modules/system/system.css 2007-04-09 08:58:03.000000000 -0500
+++ /Users/kris/drupalhead/drupal patched/modules/system/system.css 2007-04-15 17:00:01.000000000 -0500
@@ -118,13 +118,19 @@ tr.merge-up, tr.merge-up td, tr.merge-up
.nowrap {
white-space: nowrap;
}
-.pager {
+.item-list .pager {
clear: both;
text-align: center;
}
-.pager a, .pager strong.pager-current {
+.item-list .pager li {
+ background-image:none;
+ display:inline;
+ list-style-type:none;
padding: 0.5em;
}
+.pager-current {
+ font-weight:bold;
+}
.tips {
margin-top: 0;
margin-bottom: 0;