Closed (fixed)
Project:
Views Slideshow
Version:
6.x-2.0-beta2
Component:
Code
Priority:
Normal
Category:
Support request
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
11 Feb 2010 at 21:26 UTC
Updated:
18 Sep 2011 at 04:18 UTC
Jump to comment: Most recent file
Comments
Comment #1
amclin commentedThere were a couple of minor issues with the patch (didn't test after refactoring)
Here's a corrected patch.
Comment #2
dddave commentedComment #3
fletch11 commentedthanks for the patch but I'm not seeing the pager option after applying the patch. This would be a great contribution and would love to see this added.
Comment #4
amclin commentedIt should show as a 3rd type of pager you can pick. It also will only work if you're using Node as your view row type.
Comment #5
nxqamar commentedso i have never dealt with patches before. So where exactly (file) do i apply this patch? I came across this link on how to apply patches in windows http://drupal.org/node/60179.
Comment #6
nxqamar commentedfigured out with eclipse ide. dynedain thanks a lot for helping all of us.
Comment #7
psynaptic commentedSorry to be a pain but the patch doesn't follow Drupal's coding standards. Please could you re-roll the patch following the guide?
http://drupal.org/coding-standards
Comment #8
amclin commentedHere's a new patch that should follow Drupal's coding style.
Who's bright idea was it to use 2 spaces instead of tab for indentation anyways?
Comment #9
psynaptic commentedThis patch failed to apply to DRUPAL-6--2.
Comment #10
psynaptic commentedOk, I needed this feature myself and I managed to implement it with the latest dev release. Please try that and see the new options.
Comment #11
shane birley commentedI don't see the code having been added. Was this committed?
Comment #12
psynaptic commentedThis code was not added as it is not needed. The feature already exists in dev.
Comment #13
shane birley commentedI am running dev but still don't see it. I must be crazy?
Comment #14
christianpound commentedsubscribe
Comment #15
psynaptic commentedAre you using thumbnail hover?
Comment #17
amclin commentedAdding this to Thumbnail Hover node is NOT the same thing as adding titled pagers to Single Frame node. Please reopen this ticket and add the additional Pager type to Single Frame.
In my situation, I need to use Pager, not Breakouts. Breakouts in Thumbnail Hover will only display under the slides.
Comment #18
marioboro commentedHello dynedain,
thank you for the patch. Please would you be so helpful to give me some instructions?
I have applied your patch and in the settings of the slideshow i can now select "Node Titles", but they are not showed in the frontend. If i use "Numbered" it works. I have installed the 6.x.2.0-beta2 Version, because the patch is not compatible with the current version of views slideshow.
Please if you hove some free minutes to answer me. I agonize now 2 days with it.
Thank you very much!
Regards,
Mario
Comment #19
redndahead commentedMarloboro Why is Thumbnailhover not an option? This would be simple to do. Just choose the image as the main area and the title as the breakout.
Comment #20
marioboro commentedHello redndahead,
thank you for the answer. Sorry but i´m a little bit confused now. If i choose thumbnailhover instead of singleframe, i don´t have any options to set the title as the breakout.
Do i have to use the newest version of views slideshow?
Please would you be so helpful and explain it more detailed?
Thank you!
Comment #21
redndahead commentedYou need to choose Title as one of your fields and then it will show up in the settings.
Comment #22
marioboro commentedSorry, but i have choose the title as one of the fields, please see my attachement. And do i have to install the latest version?
Thank you for your understanding!
Comment #23
redndahead commentedOk do you have the thumbnailhover module enabled? If so then under slideshow settings choose thumbnailhover. Make sure you are using the latest version of views slideshow also?
Comment #24
marioboro commentedYour great, thank you a lot!
The issue was that i have tried it with an older module version and the patch. So it can´t be functional.
A last question please. Is it possible to define a tag rather a css for the title. (link, hover, active)?
Comment #25
redndahead commentedI'm not sure what you are asking can you rephrase that?
Comment #26
marioboro commentedOK, sorry ...
I´m using the titles to switch between the nodes in the slideshow.
But the titles don´t have a a tag, so i can´t define a CSS for link, hover and active state.
How i can solve this?
Thank you!
Comment #27
intyms commented@#26 marioboro
Hi, click on the title field and check "Rewrite the output of this field".
Paste this code:
Comment #28
marioboro commentedHi intyms,
that´s it! Thank you a lot for helping.
Comment #29
intyms commentedComment #31
mlconnor commentedA much easier way to do this without patching modules.
1. In views, add the field for the title, but hide the label.
2. Use a jQuery selector in the header of your view to replace the page number with the title and then hide the title.
3. Hide the title with CSS that can be added to the same header.
This is easy breezy, and doesn't require patching anything. I was able to put this together in 10 minutes.
Comment #32
mattsplatw commentedWould there be a similar approach for row style "node"? With this setting, you can't add any fields to the view. Or do I misunderstand?
Comment #33
asb commentedRe #12 vs. #13: "This code was not added as it is not needed. The feature already exists in dev."
Is this really in 6.x-2.x-dev?
Comment #34
syntheticMedia commented@#26 marioboro
But the titles don´t have a a tag, so i can´t define a CSS for link, hover and active state
can you please tell me more about this (I have followed these steps exactly)? i am trying to replicate the manner in which single frame allows you to style for the currently active slide. In other words, when the slide is active, I want a black background, and when it is not active, no background. So, basically every n seconds the black background advances, just like with singleframe.
I have done this in css:
.views_slideshow_thumbnailhover_active_teaser {
background-color:#000;
}
however the active slide does not seem to change, its always stuck at the first breakout field. In other words, the class .views_slideshow_thumbnailhover_active_teaser is only ever active on the first breakout field and does not update as with the behavior found in singleframe, (while all other slides remain with a class of .views_slideshow_thumbnailhover_div_breakout_teaser.)
Please advise, thanks!
EDIT - just saw this- does this mean that i cannot achieve active slide like with singleframe? I have tried the dev version, and fields as pager are not avialble. Furthermore, I was going to try ver3, however I do not feel as if views 3 is reliable enough yet.
Adding this to Thumbnail Hover node is NOT the same thing as adding titled pagers to Single Frame node. Please reopen this ticket and add the additional Pager type to Single Frame.
In my situation, I need to use Pager, not Breakouts. Breakouts in Thumbnail Hover will only display under the slides.
Comment #35
redndahead commentedYou can display the pager on the top or the bottom. There is an option (Display breakout fields last) Uncheck that box to make it first.
Comment #36
asb commentedSorry, the issue's title is "Display Node Titles instead of Slide numbers in the pager", and I can not see how this is "fixed" by #35.
However, I'm not entirely sure that I can imagine a use case where the node title actually would be needed in the pager (in the slideshow it's available through Views anyway).
So I'd suggest to either leave this open (and wait for a use case), or to change it to "won't fix" (if no use case exists).
Comment #37
redndahead commentedThe original issue was marked fixed. I was answering #34. For answers on the original issue read #19 #20 and #21 you need to use thumbnailhover. You can add fields using that. There are instructions in our documentation on how to use thumbnailhover. This was marked fixed for over a year and probably should stay that way.