[drupal-devel] [task] Remove theme_pager_detail()

[moshe weitzman]

Issue status update for 
http://drupal.org/node/27980
Post a follow up: 
http://drupal.org/project/comments/add/27980

 Project:      Drupal
 Version:      cvs
 Component:    base system
 Category:     tasks
 Priority:     normal
 Assigned to:  drumm
 Reported by:  drumm
 Updated by:   moshe weitzman
-Status:       patch (ready to be committed)
+Status:       patch (code needs review)

i agree with jeremy. can't we find more meaningful patches than ones
which strip themeability. I can't evaluate the patch because I don't
see a file. maybe thats a bug, or drumm just forgot.


not everyone uses our google ripoff pager. i have customized this in
the past for clients. you can't look at contributed themes and then
conclude that a theme function is not used.




moshe weitzman



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

Mon, 01 Aug 2005 20:18:04 +0000 : drumm

This themeable function is not used in core. It is used once in contrib.
The author of that module is okay with removing it.




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

Mon, 01 Aug 2005 22:25:10 +0000 : Jeremy at kerneltrap.org

The idea is to make it available for use in themes, modules, etc, to
design custom pagers.  Why exactly do you need to remove it?