Patch to add previous and next buttons

Thank you so much for this!

Hi chayner,

Again, thanks for putting this together. I noticed that when the prev/next buttons are added, they're in a ul tag that doesn't have any id or class attributes, which makes Theming a bit of a problem. However, they can be easily added by modifying lines 107 and 109 of this patch.

I'm a total noob with Drupal and patches (this was the first patch I've ever applied), and I haven't the feintest idea how to re-roll anything. So my apologies for not being much of a help.. I'll learn to do this soon (I promise!).

This patch applied clean for me and worked too. I second this much needed feature for this module.

Applied the patch - works great!

The patch can be adapted to the -dev version. The patches about the two .inc files are ok, the changes to the .js file needs a small change. The correct patch for the views_cycle.js file is:

@@ -44,6 +44,18 @@ Drupal.behaviors.viewsCycle = function(c
       return Drupal.theme('viewsCycle', thumb_item.html(), thumb_item.attr('class'), idx);
     }

+
+    var prev_string = config.params.prevnext_prev;
+    var next_string = config.params.prevnext_next;
+
+    // If the previous / next buttons have been added, create the button placeholders
+    if (config.params.prevnext_loc == 'above') {
+      cycler.before('<ul id="' + id + '-prevnext"><li><a id="'+ id + '-prev" class="prev">' + prev_string + '</a></li><li><a id="'+ id + '-next" class="next">' + next_string + '</a></li></ul>')
+    } else if (config.params.prevnext_loc == 'below') {
+      cycler.after('<ul id="' + id + '-prevnext"><li><a id="'+ id + '-prev" class="prev">' + prev_string + '</a></li><li><a id="'+ id + '-next" class="next">' + next_string + '</a></li></ul>')
+    }
+
+
     // We can't set the function from PHP because we can't specify a datatype of
     // "function" from PHP.  So we simply flag it to use the function above.
     if (config.use_pager_callback) {

It looks that the makeAnchors() function doesn't need to be modified in -dev anymore.

The patch in #5 didn't apply properly. I also think that, like the pager, the next/prev links positioning belongs as a skin property.

I've committed a modified approach that allows skins to specify whether or not they offer next/prev links. The text of the next/prev links is also themeable, so themes can tweak the HTML if desired.

Thanks all!

excuse my ignorance, I'm new to this very handy module.

Is there anyway to make the includion of the next/prev more template based, so that I can adapt the layout of the elements to create more dynamic pagers. The prev/next div and buttons are the only elements that are not handled in the template, and are not overridable in custom themes. At first I thought that this was a limit that came from the jquery.cycle project, but the html elements are hardcoded into the drupal module js file.
The benefits would be nice and it would really conform to drupal standards better (no magically appearing layout elements.) I understand that the approach taken follows in line with some other code in the module, and that my idea essentially split up some functional logic, and then magically hopes that the two parts coincide, but I still find it interesting.

I'd be willing to do it myself but I'd appreciate your advice in potential pitfalls and practical approaches.

I''m playing with it by modifying the views_cycle.js prev/next code to:

    // Determine if we're using next/previous links.
    if (config.use_prevnext_links) {
      var prevnextId = id + '-prevnext';
      config.params.next = '#' + prevnextId + ' .views-cycle-next';
      config.params.prev = '#' + prevnextId + ' .views-cycle-prev';
    }

(removing the Drupal.theme call as well of course)

and then modifying the template as follows:

<?php if ( isset($options['prevnext']['prev']) || isset($options['prevnext']['next']) ) { ?>
  <div id="<?php print $cycle_id; ?>-prevnext">
      <?php if ( isset($options['prevnext']['prev']) ) { ?><a href="#" class="views-cycle-prev"><?php print $options['prevnext']['prev']; ?></a><?php } ?>
      <?php if ( isset($options['prevnext']['next']) ) { ?><a href="#" class="views-cycle-next"><?php print $options['prevnext']['next']; ?></a>>?php { ?>
  </div>
<?php } ?>

Please advise or comment