[drupal-devel] [feature] themeable pager pieces

[Jeremy]

 Project:      Drupal
 Version:      cvs
 Component:    other
 Category:     feature requests
 Priority:     minor
 Assigned to:  Anonymous
 Reported by:  Jeremy at kerneltrap.org
 Updated by:   Jeremy at kerneltrap.org
-Status:       fixed
+Status:       patch
 Attachment:   http://drupal.org/files/issues/pager.inc_3.patch (8.85 KB)

Here's a followup patch to add classes to each of the pager pieces, and
the "...".

Jeremy at kerneltrap.org



Previous comments:
------------------------------------------------------------------------

January 26, 2005 - 23:30 : Jeremy at kerneltrap.org

Attachment: http://drupal.org/files/issues/pager.inc_1.patch (8.65 KB)

Is there anything gained from making the pager.inc pager pieces
themeable?  I'm no longer sure, but the attached patch does so.

------------------------------------------------------------------------

January 27, 2005 - 09:13 : Bèr Kessels

great! +1 from me.

But, the three dots (...) are still not thmeabnle, nor have they got a
style. The arent even wrapped in a HTML element. IMO that we sthe worst
thing of the current pager: that i could do nothing with these dots. :)

------------------------------------------------------------------------

January 27, 2005 - 09:24 : Jeremy at kerneltrap.org

Attachment: http://drupal.org/files/issues/pager.inc_2.patch (8.63 KB)

Is it sufficient to wrap them in div?  See attached.

------------------------------------------------------------------------

January 27, 2005 - 09:28 : tangent

Wrapping them in a div is only marginally better for theming purposes
without assigning a class.

------------------------------------------------------------------------

January 27, 2005 - 09:32 : Dries

Committed to HEAD.

-- 
View: http://drupal.org/node/16273
Edit: http://drupal.org/project/comments/add/16273