MMM-Carousel icon indicating copy to clipboard operation
MMM-Carousel copied to clipboard

Note: Display the same module twice on one slide

Open KristjanESPERANTO opened this issue 3 years ago • 4 comments

It took me a little time to figure this out and someone else on the forum stumbled across it too.

KristjanESPERANTO avatar Feb 23 '22 21:02 KristjanESPERANTO

Good catch for the workaround, but this is not how it's intended. It looks like I've got some flawed logic here by breaking out of the loop on a non-match: it stops looking for modules with the same name but different carouselId on the same slide config.

I'd guess you could probably also trick it by adding the modules to the slide with different carouselIds but in the reverse order they are listed in your config (bottom-up).

I will leave this PR open for now as a reminder, but I'm going to try and revisit the logic when I get a chance.

shbatm avatar Feb 23 '22 21:02 shbatm

And I thought it was supposed to be like this and I'm just having trouble understanding it 😅

KristjanESPERANTO avatar Feb 23 '22 22:02 KristjanESPERANTO

Nope, just bad logic. It was intended to allow the same module with different configs to be shown on different slides, but didn't take into consideration the same module, different configs, same slide.

shbatm avatar Feb 23 '22 23:02 shbatm

See the example in #36. CarouselId section-2-detail demonstrates two entries with the same id

paulsp avatar May 31 '22 11:05 paulsp