[feat] Show Multiple Images as a Carousel#302
Conversation
|
Yay, web components! I took a look, good stuff! Noticed a few minor things:
Screenshot showing missing margins and the caption issue: I’m also wondering whether we should indeed crop the images to a fixed aspect ratio … this will cause issues with portrait images, for sure. On Publix we have a similar slider and the images all get the same height, while keeping their original ratio. Maybe this would be a better fit?
|
|
Thanks for being thorough, @harryfk! I've fixed the spacing and width issues and also modified the transform to use the asset's original aspect ratio:
|
|
Woah, that looks ace! I’ll quickly check it locally. |
harryfk
left a comment
There was a problem hiding this comment.
Coool, this looks much better now! I’ve tested it locally and it all looks great to me. In my RSS reader the markup structure with slides as a list and then also another list in the figcaption is a bit odd, but also makes sense … I’m leaning towards keeping it like this.
Also works great as a standalone module! All good from my end.



This PR enables editors to display multiple images in a carousel.
Changes
imagesfield and auto-generated title (e.g. "3 Images")modulesmatrix field and thetextEntryTypesrich-text field so editors can insert galleries anywhere<gallery-carousel>custom element powered by Splideimport()to keep the initial JS bundle small(pipe-separated), and optional secondary rich-text caption
columnsrich-text layout