fix: quick links widget overlap with plane ai promo on mobile#9339
fix: quick links widget overlap with plane ai promo on mobile#9339Ayusman-Singhal wants to merge 1 commit into
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughModified spacing and flex layout classes in the DashboardQuickLinks header markup, changing container margin to a responsive value and adjusting the header row to wrap with additional gap and margin. ChangesQuick Links Layout Fix
Estimated code review effort: 1 (Trivial) | ~2 minutes Related issues: Fixes "+ Add Quick Link" text overlapping Plane AI promotional text on mobile browser ( Suggested labels: bug, frontend, mobile Suggested reviewers: N/A Poem: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ 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 |
There was a problem hiding this comment.
Pull request overview
Fixes a mobile layout issue on the Home dashboard where the Quick Links widget header could overflow/bleed into the Plane AI promo section on narrow viewports.
Changes:
- Adds extra bottom spacing on small screens for the Quick Links widget container (
max-sm:mb-6). - Allows the Quick Links header row (title + “Add” button) to wrap and adds spacing between wrapped rows (
flex-wrap+gap-2).
Description
Quick Links widget text overflowed into Plane AI promo section below on mobile viewports (<640px). The "Add quick Link" button (3 words) + "Quicklinks" title in
justify-betweenflex had no wrap behavior, causing text collision. Insufficient bottom margin made the overlap visible.Changes
apps/web/core/components/home/widgets/links/root.tsxtwo class changes:mb-2→mb-2 max-sm:mb-6: triple bottom clearance on small screens, preventing bleed into next widgetflex→flex flex-wrap gap-2: title and button wrap to separate lines with proper spacing instead of collidingType of Change
Screenshots and Media
Test Scenarios
References
Closes #9084
Summary by CodeRabbit