Last slide is aligned to the left but should be aligned to the right / Track width

[Romcol]

Hi,

Having multiple items per page on the carousel we would expect the "next arrow" not to be clickable when the last item is shown. Currently, the arrow is clickable and also the carousel is draggable to the right as long as the last item is not aligned to the left. I think, as seen with other libs, we shouldn't be able to slide that far down the track.

To reproduce :

Remove infinite on this example : https://brainhubeu.github.io/react-carousel/docs/examples/multipleItems .

Thank you for the lib :fire:

---

IssueHunt Summary

Backers (Total: $0.00)

Submitted pull Requests

• #536 [FIX] Limit reachable slide in Carousel

---

Become a backer now!

Or submit a pull request to get the deposits!

Tips

• Checkout the Issuehunt explorer to discover more funded issues.
• Need some help from other developers? Add your repositories on IssueHunt to raise funds.

[piotr-s-brainhub]

@Romcol

Thanks for reporting this.

I also noticed that, I don't like this behavior but for me, it's a minor bug.

You're more than welcome to open a PR to fix that.

You can also invite your colleagues to like (👍) this issue so more 👍it has, it's more likely to be fixed by us.

[Romcol]

Hi,

@piotr-s-brainhub Sure, I'll try to make a PR. Have you got any pointers on how to implement this?

I've seen that there's something such as an activeSlide which is aligned to the left. We can prevent the user from selecting a slide past a certain slide maybe?

[piotr-s-brainhub]

@Romcol

I'm not sure about an exact place in the code but generally, I guess, we should calculate whether there's a next slide to display and according to that allow or not to move right.

[issuehunt-oss[bot]]

@piotr-s-brainhub has funded $7.00 to this issue.

---

• Submit pull request via IssueHunt to receive this reward.
• Want to contribute? Chip in to this issue via IssueHunt.
• Checkout the IssueHunt Issue Explorer to see more funded issues.
• Need help from developers? Add your repository on IssueHunt to raise funds.

[meysamizadmehr]

I can reproduce it on 1.19.17. Is it released?

[piotr-s-brainhub]

@meysamizadmehr

not yet, if it was fixed, the GitHub issue would be closed

[piotr-s-brainhub]

@Romcol

I leave the brainhubeu organization so I must remove the IssueHunt funding as it's only me who's able to manage these funds.

Moreover, the team decided on Thursday about stopping using IssueHunt.

[issuehunt-oss[bot]]

@piotr-s-brainhub has cancelled funding for this issue.(Cancelled amount: $7.00) See it on IssueHunt

[piotr-s-brainhub]

@Romcol

As your work is already on https://github.com/brainhubeu/react-carousel/pull/537, you can try reaching out @RobertHebel , @humbak or @Lukasz-pluszczewski or another brainhubeu organization member in order to be rewarded on IssueHunt.

[Romcol]

Hi,

Cool. How should I reach you? Thanks

[RobertHebel]

@Romcol We are working on restoring funding for already opened PRs. On the next Thursday, I will be able to re-assign money for this issue. Is it ok for you to wait for the funding one week?

[RobertHebel]

Issuehunt for this issue will be restored tomorrow

[issuehunt-oss[bot]]

@roberthebel has funded $7.00 to this issue.

---

• Submit pull request via IssueHunt to receive this reward.
• Want to contribute? Chip in to this issue via IssueHunt.
• Checkout the IssueHunt Issue Explorer to see more funded issues.
• Need help from developers? Add your repository on IssueHunt to raise funds.

[issuehunt-oss[bot]]

@roberthebel has cancelled funding for this issue.(Cancelled amount: $7.00) See it on IssueHunt