Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
A problem arises when only one image is attached to the node, the slideshow transitions between the same image creating a slow flash effect. It would be handy if the slideshow behavior only applied to nodes with multiple images.
Comment | File | Size | Author |
---|---|---|---|
#13 | 1341194-13_views_galleriffic_m-a.patch | 4.67 KB | alexandrezia |
#12 | 1341194-12_views_galleriffic_m-a.patch | 4.62 KB | Marc-Antoine |
#11 | 1341194-11_views_galleriffic_m-a.patch | 4.64 KB | Marc-Antoine |
#10 | 1341194-9_views_galleriffic_m-a.patch | 4.86 KB | Marc-Antoine |
#7 | 1341194-7_views_galleriffic_m-a.patch | 4.87 KB | Marc-Antoine |
Comments
Comment #1
letrotteur CreditAttribution: letrotteur commentedLooking for this kind of behavior too. Anyone has ideas on how to accomplish this?
Comment #2
Marc-Antoine CreditAttribution: Marc-Antoine commentedHere's my approach:
If a single picture gallery is detected, change the main picture's container from 'a' to 'div' and remove the thumbnail pager (no need to show a single thumbnail).
tell me what you think, is that what you had in mind?
Comment #3
Marc-Antoine CreditAttribution: Marc-Antoine commentedchanging the status
Comment #4
Marc-Antoine CreditAttribution: Marc-Antoine commentedHere's a better handling of the number of pictures in the .js, you should use this patch
Comment #5
acouch CreditAttribution: acouch commentedI don't think it is better to have no thumbnail than one.
However, I would accept this if the logic for the display was moved from the template to the preprocess function and there was a setting in the Gallery settings that didn't hide the thumbnail by default. I made some changes recently so re-rolled in case you want to keep working on this.
Comment #6
Marc-Antoine CreditAttribution: Marc-Antoine commentedIndeed the best handling would be to enable such an option, I will see if I have some time to work on this. Thank you for answering.
Comment #7
Marc-Antoine CreditAttribution: Marc-Antoine commentedHey there, I worked on a new patch for you, it adds a 'hide single picture's thumbnail' option within the view's format.
The default state of this option is 'no' (therefore not hiding a single thumbnail) as you requested.
Also, the single thumbnail is still clickable, I don't know what you think about that behavior, and I didn't want to change it until you said something about it.
I attached a screenshot (sorry for the french interface) of the option within it's form.
Have a good day!
*edit*
This option still acts on the template though, since I haven't found a clever way of preventing the gallery from displaying it's thumbnail elsewhere.
Comment #8
Marc-Antoine CreditAttribution: Marc-Antoine commentedchanging the status
Comment #9
acouch CreditAttribution: acouch commentedMarc, your english is great and approach sound. It will be a bit before I get to update the module but will try and include this when I can.
Comment #10
Marc-Antoine CreditAttribution: Marc-Antoine commentedThank you, I'm eager to see what you have to say about the patch.
Here's a new version (minor bug corrected). Also I realized most of your options' boolean values are in fact string types, while mine are not (true instead of 'true'), perhaps I should rewrite the patch for the sake of consistency?
See you later!
Comment #11
Marc-Antoine CreditAttribution: Marc-Antoine commentedCorrected a corner case bug
Comment #12
Marc-Antoine CreditAttribution: Marc-Antoine commentedRerolled the patch to be applied against the 7.x-1.1
Comment #13
alexandrezia CreditAttribution: alexandrezia commentedRerolling the patch to apply against 7.x-1.x bc9ae19a8f0343c897ab34b24ea6c96da2a953d2