Closed (fixed)
Project:
Views Slideshow: Dynamic Display Block
Version:
6.x-2.x-dev
Component:
User interface
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Created:
1 May 2011 at 12:09 UTC
Updated:
28 Sep 2011 at 12:21 UTC
Jump to comment: Most recent file
Comments
Comment #1
raincloud commentedI have the same problem, only I use the 7.x-1.0-beta1 and Drupal 7. After unchecking the "Use pager" option in ddblock configuration, the pager displays as a simple html unordered list. So it is no more themed (I use upright10p) but is still in html.
Comment #2
ppblaauw commentedFor view_slideshow_ddblock will need to add $pager_toggle variable to the available settings for the slideshow themes.
For ddblock drupal 7 the $pager_toggle variable is already available to the slideshow themes.
For both the ddblock module and the views_slideshow_ddblock module:
In the content_[theme_name].tpl.php file of all themes a check needs to be added if the $pager_toggle variable is checked
For example:
needs to be changed into
Will apply patches for this soon
Comment #3
raincloud commentedThis did the trick (the change in *tpl.php) ! In the meantime, another temporary solution was to add at the bottom of css file:
.none, .number-pager-pre-top {
display:none;
}
Comment #4
ppblaauw commentedAdded pager_toggle variable to dev version and version 2.0
$pager_toggle variable can now be used in slideshow themes to disable the pager.
Comment #6
sp3boy commentedI have installed 6.x-2.x-dev and changed the views-slideshow--ddblock-cycle-block-content-vsd-upright-60p.tpl.php as instructed. No pager thumbnails are output, however the main slides remain invisible also. I think this is due to a javascript error that is "myScrollable is undefined .../sites/default/modules/contrib/views_slideshow_ddblock/js/views_slideshow_ddblock.js?r Line 285"
Inspecting the $pager_toggle theme template variable in the .tpl.php, it is always null irrespective of how the view is configured, suggesting a problem with the preprocessing?
Comment #7
ppblaauw commentedIf you don't want to use a pager, you can use the vsd-upright-10p, vsd-upright-20p, vsd-upright-30p, vsd-upright-40p and vsd-upright-50p slideshow themes.
The vsd-upright-60p uses a scrollable pager and the javascript code wants to target the scrollable pager.
That is why you get the javascript error.
Hope this helps you further, please let me know.
Comment #8
sp3boy commentedAre you saying the option to disable the pager will cause an error unless I use a template that doesn't have a pager anyway? That implies that the disable option has no practical use.
My client wanted the upright-60 layout, with the row of circular icons but simply without the row of thumbnail images, so the presence of an option on the View config to disable the thumbnail pager appeared to be ideal.
Comment #9
sp3boy commentedFor what it's worth I think the attached patch resolves the problem whereby the $pager_toggle theme template variable in the .tpl.php is always null.
Comment #10
ppblaauw commentedSP3boy
Thanks for the patch
Committed to dev version.