fix: prevent leaderboard header overflow on mobile - #38
fix: prevent leaderboard header overflow on mobile#38gauri9368gupta-maker wants to merge 1 commit into
Conversation
|
@gauri9368gupta-maker Thanks for the PR, man. The fix works, but it's very minimal. I was expecting a more complete responsive solution for this issue. Please improve it by:
Please push the requested changes within the next 24 hours. I'll review it again after the update. |
| white-space: normal; | ||
| max-width: 100%; | ||
| } | ||
|
|
There was a problem hiding this comment.
The current changes still rely only on overflow-wrap and word-break to prevent the overflow. This doesn't fully address the requested review changes.
Please update the PR by:
- Use responsive font sizing (e.g.
clamp()). - Add media queries to improve the layout across 320px, 360px, 393px, and 430px screen widths.
- Adjust spacing/alignment where necessary so the header remains readable without relying only on text wrapping.
- Expand the implementation into a more complete responsive solution.
Once these changes are made, I'll review the PR again.
|
This PR is being closed due to inactivity. The requested review changes were not addressed within the given timeframe. Please complete the requested improvements and submit a new PR when it's ready. Thank you for your contribution. |
Description
Fixed the mobile overflow issue in the leaderboard header.
Fixes #35
Changes Made
.terminal-titleTesting
Screenshots
Attached screenshots for reference.
Note: The GitHub live page and localhost page may appear slightly different due to deployment/cache differences. The issue was reproduced and fixed locally, and the PR contains the responsive overflow fix.