The other day we wanted to add a carousel header item for the climate strikes, but there were already 4 items, which is the maximum (hard-coded here if I’m not mistaken: https://github.com/greenpeace/planet4-plugin-blocks/blob/f0f454b15d0a4d0be61ecbc0a82363c581253161/classes/controller/blocks/class-carouselheader-controller.php#L68) so we had to remove one to make room.
I understand we don’t want dozens of entries in there, but I was wondering if there was a real reason to limit to 4, or if perhaps 5 would have been OK.
So this is the feature request: allow for an arbitrary amount of carousel header items.
== UPDATE 17/11/2020==
The P4 team decided to decline this idea as our latest analysis shows that most users move on around 3-4 slides.
Discussion
The reason for the carousel items set to four is because of load times. You notice sometimes you have an off centre action that occurs when the images are not properly formatted or too large.
I think one solution might be to set the system to give an error message if the photos are not saved at the right dimension and size, which is as small as possible... less the 300k
Another solution might be to load only the first item in the caroussel on page load and defer the other items to load asynchronously.
The P4 team decided to decline this idea as our latest analysis shows that most users move on around 3-4 slides.