Fix: prevent vertical overflow on initiative card hover#114
Conversation
|
@KitsuneKode is attempting to deploy a commit to the WeMakeDevs' projects Team on Vercel. A member of the Team first needs to authorize it. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (1)
📝 WalkthroughWalkthroughA single Tailwind CSS class change in ChangesInitiatives Carousel Spacing
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~1 minute Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 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 |
Summary
Fixes vertical overflow in the Initiatives carousel when hovering over a card and the "Apply" button appears.
What changed
Increased bottom padding on the carousel scroll container from
pb-4topb-10inInitiatives.tsx, giving the hover-revealed Apply button enough space so it no longer spills outside the container.Problem
On hover, the Apply button slides into view at the bottom of each initiative card. With only
pb-4(16px) of padding, the button extended past the scroll area, causing a vertical scrollbar and layout shift.Solution
Reserve more space below the card row (
pb-10/ 40px) so the Apply button can appear on hover without triggering vertical overflow in the horizontal scroll container.Screenshots
Before
After
Test plan
Summary by CodeRabbit