Based on comments from http://drupal.org/node/1168658, we would like the pager fields to be links, so that clicking on them can take us to that item in the cycle.

Comments

jenlampton’s picture

StatusFileSize
new1.72 KB

Some of this patch includes the jQuery added by Paul.B in the other issue, but that was not sufficient for me, I also had to make the items into links, so my mod's added as well.

bleen’s picture

StatusFileSize
new1.71 KB

this is the identical patch as in #1 with some whitespace problems removed

anniegreens’s picture

I've discovered an issue with the use case where multiple items are shown on one slide, an option of Views Slideshow. The current implementation does not take this variable into account. So, for example, if this option is set with three (3) items per slide with a total of three (3) slides, the pager will output nine (9) items but only 1, 2, and 3 are actually clickable.

The attached patch includes changes to support this correctly, as well as the changes for the pager to be clickable from above.

anniegreens’s picture

New patch using l() for clickable pager and javascript:void(0) instead of # to avoid page jump when clicking through pager.

nickdickinsonwilde’s picture

Issue summary: View changes
Status: Active » Fixed

Applied patch to version in Views Slideshow 7.x-3.7

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.