UI: fix example prompts overflow and mobile icons not reaching screen edge#719
Conversation
… edge 1. Example prompts overflow on mobile: - Added overflow-hidden to container to prevent horizontal overflow - Added whitespace-normal, break-words, and max-w-[calc(100vw-2rem)] so long prompts break to a new line instead of extending past the screen - Used gap-1.5 between icon and text instead of mr-2 for consistent spacing - Icons use flex-shrink-0 to prevent squishing 2. Mobile icons not reaching end of screen: - Removed min-width: max-content which caused content to shrink and leave empty space on the right - Set width: 100% to ensure the icons bar fills the full screen width - Changed from space-between to space-evenly for consistent equal distribution - Removed the tablet override media query (768px) that was overriding with gap: 0 and space-between - Set consistent gap of 8px across all mobile/tablet sizes
|
Someone is attempting to deploy a commit to the QCX-MAIN Team on Vercel. A member of the Team first needs to authorize it. |
Qodo reviews are paused for this user.Troubleshooting steps vary by plan Learn more → On a Teams plan? Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center? |
|
ngoiyaeric seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account. You have signed the CLA already but the status is still pending? Let us recheck it. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (2)
📜 Recent review details⏰ Context from checks skipped due to timeout. (1)
🔇 Additional comments (2)
WalkthroughResponsive layout classes now distribute mobile icon-bar items evenly and constrain empty-screen example messages with overflow handling, wrapping, and responsive width behavior. ChangesResponsive layout adjustments
Estimated code review effort: 2 (Simple) | ~10 minutes Possibly related PRs
Suggested labels: Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 ESLint
ESLint install failed. For unrecoverable errors, disable the tool in CodeRabbit configuration. 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 |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Summary
This PR fixes three UI issues:
1. Example Prompts Overflow Past Screen Edge on Mobile
File:
components/empty-screen.tsxLong example prompts (e.g., "How does climate change affect our experience?") were extending past the screen width on mobile. Fixed by:
overflow-hiddento the container to prevent horizontal overflowwhitespace-normal,break-words, andmax-w-[calc(100vw-2rem)]so prompts break to a new line instead of overflowinggap-1.5between icon and text (instead ofmr-2) for consistent spacingflex-shrink-0to prevent squishing2. Mobile Icons Not Reaching End of Screen
File:
app/globals.cssThe mobile icons bar was leaving empty space on the right side of the screen. Fixed by:
min-width: max-contentwhich caused the content to shrink and leave a gapwidth: 100%to ensure the icons bar fills the full screen widthspace-betweentospace-evenlyfor consistent equal distributiongap: 0andspace-betweengap: 8pxacross all mobile/tablet sizes3. SW Update Notification Popup Improvements
File:
components/sw-update-notification.tsxThe service worker update notification popup had several issues:
bottom-leftso the popup appears on the left side of the screenInfinity(permanent) to 15 seconds so the toast auto-dismisses instead of staying forever. Also added ahasShownstate to prevent the popup from showing more than once per session