feat: add looping functionality to slideshow for seamless experience#1100
Conversation
|
Warning Rate limit exceeded
⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. 📝 WalkthroughWalkthroughThe changes add looping functionality to the slideshow feature by introducing a handler that resets the image position to the start and updating the useSlideshow hook to conditionally invoke looping logic when reaching the final image. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
Hi maintainers , I am interested in contributing to this issue can you please assign it me? |
rahulharpal1603
left a comment
There was a problem hiding this comment.
Thanks @monu808!
@rahulharpal1603
Fixes: #1090
Description
Fixed the slideshow stopping at the last image instead of looping continuously.
Changes
Updated useSlideshow hook to support looping via optional onLoopToStart callback
Added handleLoopToStart in MediaView.tsx to reset to first image
Testing
Open any image in full-screen viewer
Click "Slideshow" button
Wait for slideshow to reach last image
Slideshow now loops back to first image
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.