beyonk-group / svelte-carousel

A super lightweight, super simple Carousel for Svelte 3
MIT License
213 stars 38 forks source link

Incorrect number of dots in example #72

Open bobspace opened 3 years ago

bobspace commented 3 years ago

The Carousel example on svelte.dev has four items but only two dots.

Is that an issue with the Carousel or svelte.dev?

image
raghav-kukreti commented 3 years ago

The dots don't really demonstrate the number of items but the number of slide actions available to view all items. That's what I've been looking at them as. On looking at the source, this line helps understand it's purpose $: totalDots = controller ? Math.ceil(controller.innerElements.length / currentPerPage) : [];