fix(dashboard): Prevent horizontal overflow and improve layout on small screens - #8309
Conversation
|
@nishtha-agarwal-211 is attempting to deploy a commit to the jhasourav07's projects Team on Vercel. A member of the Team first needs to authorize it. |
|
This pull request effectively resolves the issue of horizontal overflow and layout shifts on smaller screens, as outlined in issue #8302. The changes made to the grid styling and layout will enhance the user experience on devices with limited screen width. Additionally, similar past decisions have focused on improving responsive layouts and ensuring that components render correctly across various devices. This aligns well with our ongoing efforts to maintain a high-quality user interface. Thank you for your contributions, and I look forward to your updates! |
📦 Next.js Bundle Size Report (Gzipped Sizes)
📊 Summary of Totals
|
Aamod007
left a comment
There was a problem hiding this comment.
Fixing horizontal overflow on small screens with CSS tweaks in DashboardClient and StatsCard. CI failing, please fix.
Labels: level:beginner (2 files, 9+9), quality:clean, type:bug, mentor:Aamod007
Aamod007
left a comment
There was a problem hiding this comment.
Re-reviewing after noting the only CI failure is Vercel authorization (not code-related). All other checks (Format · Lint · Typecheck · Test, Production Build, CodeQL) pass. Approving — the Vercel deploy auth is a repo-level config issue, not a code problem.
Description
Fixes #8302
Changes Summary
DashboardClient.tsx): Replacedgrid-cols-2base grid styling withgrid-cols-1 sm:grid-cols-2 md:grid-cols-4on screens< 640px(specifically viewports< 375pxlike Galaxy Fold or iPhone SE) to stack action buttons vertically without overflowing the viewport.DashboardClient.tsx): Enforcedw-full max-w-full overflow-x-hiddenon the#dashboard-rootelement to prevent horizontal document scrolling.StatsCard.tsx):p-4 sm:p-6.min-w-0 flex-1andbreak-wordstext wrapping to metric text elements to prevent truncation clipping on small screens.shrink-0to the icon wrapper.Pillar
Visual Preview
Verified responsive column stacking and document width containment on 360px viewport width in Chrome DevTools.
Checklist before requesting a review:
CONTRIBUTING.mdfile.localhost:3000/api/streak?user=YOUR_USERNAME).npm run formatandnpm run lintlocally and resolved all errors (CI will fail otherwise).feat(themes): ...,fix(calculate): ...).README.mdif I added a new theme or URL parameter.