fix: make Employer Assessment page responsive on mobile/narrow screens - #27
Conversation
Fixes Yuvraj-Sarathe#4 Root cause: the 3-column layout (h-dvh + overflow-hidden on <main> + overflow-y-auto per pane) was designed for desktop's side-by-side independent scrolling. On mobile the columns stack (grid-cols-1), but overflow-hidden clipped everything past viewport height instead of scrolling, so the 'Consult the AI' question bar (last item in the right column) became unreachable. Fix: scope the fixed-height/clip/independent-scroll behavior to lg: and above, so mobile falls back to normal page scroll. Also added lg:min-h-0 to fix grid items' default min-height:auto blocking their own overflow scroll on desktop, and gave the report panel a sane min-height on mobile instead of h-full collapsing to 0.
|
@ishhwarrii is attempting to deploy a commit to the Yuvraj Sarathe's projects Team on Vercel. A member of the Team first needs to authorize it. |
📝 WalkthroughWalkthroughThe assessment page now uses responsive viewport heights, large-screen-only internal scrolling, adjusted sidebar sizing, and responsive detailed output panel constraints. The TypeScript incremental build artifact is also ignored. ChangesResponsive assessment layout
Repository hygiene
Estimated code review effort: 1 (Trivial) | ~5 minutes Possibly related PRs
Suggested reviewers: 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ Finishing Touches 💡 1🛠️ Fix failing CI checks 💡
🧪 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 |
|
🧙 Sourcery has finished reviewing your pull request! Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
Yuvraj-Sarathe
left a comment
There was a problem hiding this comment.
Make sure your changes align with the linting tests.
You can check why this is failing by either visiting logs or just simply switching to the Files Changed tab in the PR.
|
@ishhwarrii since this is a UI PR, please add screenshots of verification, and I can merge the PR right away! |
|
@Yuvraj-Sarathe Have uploaded the screenshots! please checkout. |
🔗 Related Issue
Closes #4
📝 Description of Changes
This PR enhances the Employer Assessment page for mobile devices and smaller screen widths by improving its responsive layout.
What's changed
lg) only.lg:min-h-0to grid items to ensure desktop panels maintain proper independent scrolling.These changes preserve the desktop experience while providing a smoother and more intuitive experience on narrow screens.
🏷️ Proposed Labels
📂 Core Files Changed
📸 Verification & Screenshots
UI/UX (User Interface / User Experience - how it looks and feels):
CI/CD (Continuous Integration / Continuous Deployment - the automated build/test pipeline):
🤖 AI Assistance Declaration
Did you use an AI tool to write or assist with this code OR Pull Request?
This PR focuses exclusively on responsive layout improvements. Desktop behavior has been preserved, while mobile users can now scroll through the page normally and access all content without layout issues.
✅ The "I Swear I Didn't Break Anything" Pledge
Summary by Sourcery
Improve the Employer Assessment page layout to behave responsively on mobile and narrow screens while preserving the desktop experience.
Enhancements:
Summary by CodeRabbit
Screenshots:


