Closed abdulbasithqb closed 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 | ❌ Failed (Inspect) | Mar 18, 2024 6:11am |
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 95.53%. Comparing base (
7461173
) to head (3c86b44
). Report is 202 commits behind head on main.:exclamation: Current head 3c86b44 differs from pull request most recent head e93f133. Consider uploading reports for the commit e93f133 to get more accurate results
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
@coderabbitai review
This update introduces a straightforward yet effective change to the Carousel component, specifically by adding an inline style to enable horizontal scrolling (overflowX: 'scroll'
). This adjustment aims to enhance the component's usability, particularly addressing an initialization bug that affected the Carousel's functionality upon first page load.
File Path | Change Summary |
---|---|
src/components/Carousel/... |
Added inline style overflowX: 'scroll' |
Objective | Addressed | Explanation |
---|---|---|
Carousel init bug (#1283) | ✅ |
🐇 "In the realm of code, a tiny fix we sew,
🎠 Carousel now scrolls, with a gentle flow.
A bug once lurked, in the page's first glow,
❌ Now vanquished it lies, with nowhere to go.
🌟 To developers and users, a smoother show."
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?
@abdulbasithqb please, don't use merge to sync with main, instead use rebase. Revert the merge and rebase, otherwise it won't be possible to accept the PR.
@abdulbasithqb please, don't use merge to sync with main, instead use rebase. Revert the merge and rebase, otherwise it won't be possible to accept the PR.
closed this PR and raised a new one PR: #1305
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