The 2026 Drupal Business Survey is open. Your response will help shape the future of the Drupal business ecosystem.I used a lot colorbox module, but lack of responsive handling force me to switch to photoswipe module :) But one feature I'm missing - set different image style for first image. I made a patch based on colorbox implementation.
| Comment | File | Size | Author |
|---|---|---|---|
| #21 | photoswipe-set-different-first-image-style-2711625-21.patch | 3.6 KB | anybody |
| #18 | photoswipe-set-different-first-image-style-2711625-18.diff | 3.57 KB | anybody |
| #11 | photoswipe-set-different-first-image-style-2711625-5.patch | 3.31 KB | Anonymous (not verified) |
| #4 | photoswipe-set-different-first-image-style-2711625-4.patch | 2.91 KB | parisek |
| style.png | 830.94 KB | parisek |
Comments
Comment #2
parisekComment #3
parisekComment #4
parisekRemoved debug code
Comment #5
parisekComment #8
ParisLiakos commentedthank you!
Comment #9
ParisLiakos commentedah
Comment #10
ollie222 commentedJust a quick comment to say thank you for this in 7.x-2.dev, it's a fantastically useful feature to have.
Comment #11
Anonymous (not verified) commentedComment #12
Anonymous (not verified) commentedAttempt at porting the D7 patch to D8. Works nicely for me. Please review!
Comment #13
chucksimply commented#11 worked for me in D8, although breaks if the second image option is set to Hide(do not display image).
Comment #14
m.abdulqader commented#11 worked in D8.
Comment #15
thomas.frobieter+1 for this. This was a very usefull Colorbox formatter feature.
Comment #16
anybody#11 makes a lot of sense and seems clean to me. @ChuckSimply can you please explain how it breaks with hide set? What's the expected vs. result?
I can't see a reason in code. The theme function doesn't even know about "hide".
Comment #17
anybodyJust a little addition for #11: A check if isset($variables['delta']) should be added, otherwise we get a php notice if delta doesn't exist.
Comment #18
anybodyCorrected patch attached. I think we can leave this RTBC because only the isset was added and the patch is very short.
Comment #19
anybodyWe're using this in production on several sites and it works great. A really helpful feature making this module even more powerful.
Can we get this into the dev branch, sun? What do you think? As you can see it's in 7.x since a long period of time already and would make 8.x even more complete.
Comment #20
anybody@sun: Is there a possibility to get this into the next beta5?
Comment #21
anybodyRerolled the patch against latest dev. Please review and RTBC. No code changes, simple reroll.
Comment #22
thomas.frobieterPatch applies cleanly and works as expected.
Comment #23
anybodyPLANNED for the next dev and stable release.
Comment #25
anybody