Open abdulbasithqb opened 3 months ago
Run & review this pull request in StackBlitz Codeflow.
The latest updates on your projects. Learn more about Vercel for Git ↗︎
Name | Status | Preview | Comments | Updated (UTC) |
---|---|---|---|---|
flowbite-react | ✅ Ready (Inspect) | Visit Preview | 💬 Add feedback | Mar 18, 2024 6:20am |
The recent update introduces a style
property to the Carousel
component, specifically setting overflowX
to 'scroll'. This change aims to enhance user interaction with the Carousel by enabling horizontal scrolling, potentially addressing issues related to Carousel initialization and usability on mobile devices.
File Path | Change Summary |
---|---|
src/components/.../Carousel.tsx |
Added style property with overflowX: 'scroll' . |
Objective | Addressed | Explanation |
---|---|---|
Carousel does not work on first page load [#1283] | ❓ | It's unclear if the overflowX: 'scroll' directly addresses the initialization issue without further context on the bug's cause. |
Allow scrolling through the page on mobile devices when Carousel is the only visible element [#1280] | ✅ | |
Investigate scrolling limitation on iOS devices [#1280] | ❓ | The change might impact scrolling on iOS, but without specific testing or feedback, it's unclear if it fully resolves the issue. |
overflowX: 'scroll'
may indirectly address scrolling issues on mobile devices, making this issue potentially relevant to the PR. The change could improve usability on iOS devices, meeting some of the objectives outlined.In a world of static views and clicks,
A tiny change makes the Carousel ticks.
With a scroll to the left and a scroll to the right,
Smoothly gliding, a delightful sight.
🐇 CodeRabbit toiled by the moon's soft gleams,
Crafting experiences beyond mere dreams.
"Scroll on," it whispers, "to see wonders untold,
In the magical Carousel that now smoothly scrolls."
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 95.53%. Comparing base (
7461173
) to head (11677ec
). Report is 202 commits behind head on main.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
fix: carousel init bug
Changes:
Added Overflow Props to ScrollContainer
Why:
fixes: #1283
Screen-record:
https://github.com/themesberg/flowbite-react/assets/134603758/f40425df-cddf-4beb-b5bd-a488dc0806a6
Summary by CodeRabbit
Summary by CodeRabbit