Skip to content
This repository has been archived by the owner on May 5, 2022. It is now read-only.

Incorrect number of dots in example #72

Open
bobspace opened this issue Jul 1, 2021 · 1 comment
Open

Incorrect number of dots in example #72

bobspace opened this issue Jul 1, 2021 · 1 comment

Comments

@bobspace
Copy link

bobspace commented Jul 1, 2021

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
Copy link

raghav-kukreti commented Jul 21, 2021

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) : [];

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants