[drupal-devel] [feature] Paging support for long page nodes

[robertDouglass]

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

 Project:      Drupal
 Version:      4.6.3
 Component:    page.module
 Category:     feature requests
 Priority:     normal
 Assigned to:  Anonymous
 Reported by:  scutari
 Updated by:   robertDouglass
 Status:       patch (code needs review)

Has anyone considered asking The Onion if they would be able to share
the code that does that? It might be that they wrote custom Drupal code
just for the purpose. Seeing as they are using Drupal to make one of the
best looking news sites on the web, they might be receptive to giving
back to the community.




robertDouglass



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

Wed, 31 Aug 2005 10:36:10 +0000 : scutari

Attachment: http://drupal.org/files/issues/submitted_page_0.patch (1.84 KB)

This is a *beta* support for paging (splitting long page nodes into
smaller chunks) using a  <!-- page --> tag.


Even in this beta stage it works flawlessly in my drupal 4.6.3.


Preview of splitted nodes is broken without some other hacks; I'll
provide them as patches if this feature request is deemed interesting.




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

Wed, 31 Aug 2005 11:08:21 +0000 : lennart

I find this to be an important and interesting feature!


Best wishes,
Lennart




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

Wed, 31 Aug 2005 11:26:03 +0000 : scutari

Attachment: http://drupal.org/files/issues/submitted_node_0.patch (692 bytes)

I'm very happy it's useful!


With this additional patch (it's against node.module) the <!-- page -->
tag is changed to <!-- pg --> in node_preview so that paging is not
triggered. The whole page in displayed instead.


Yes, it's a dirty hack, and comments are welcome if anybody knows how
to do it in a clean way.


Regards
   Marco




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

Wed, 31 Aug 2005 12:24:47 +0000 : sgwealti

The onion website just switched to drupal and they seem to have a pretty
good pager for their nodes.  I wonder how they did it.




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

Wed, 31 Aug 2005 12:32:23 +0000 : Bèr Kessels

IMO this really needs to live in a module.




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

Wed, 31 Aug 2005 12:54:49 +0000 : scutari

It can be done in a module using a couple of hooks, but in my opinion
it's overkill for such a small change. I think the proper place for
such a feature is core.


A new core module may be fine, too, but I would like to hear some other
comments before I
make up my mind.




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

Wed, 31 Aug 2005 14:30:16 +0000 : robertDouglass

It would need a menu item to intercept the node/nid/pagenumber param,
and a filter to split the pages at a configured length. Ber is right
that this should start out as a module, and then when it has proven
itself, get integrated into core. I definitely want to see it happen
but won't get around to programming it myself anytime soon.