tweaks on hover of leaderboard badges - #2137
steam-bell-92 wants to merge 1 commit into
Conversation
|
@steam-bell-92 is attempting to deploy a commit to the recode Team on Vercel. A member of the Team first needs to authorize it. |
|
Thank you for submitting your pull request! 🙌 We'll review it asap. The estimated time for response is 5–8 hrs. In the meantime, please provide all necessary screenshots and make sure you run - npm build run , command and provide a screenshot, if it's a Video -LEGEND, which helps speed up the review. If you have questions, reach out to LinkedIn. You can see in our Leaderboard in 20 min from now!😊 Note: I maintain the repo issue every day twice at 8:00 AM IST and 9:00 PM IST. If your PR goes stale for more than one day, you can tag and comment on this same issue by tagging @sanjay-kv. We are here to help you on this journey of open source. Consistent 20 contributions are eligible for sponsorship 💰 🎁 check our list of amazing people we sponsored so far: GitHub Sponsorship. ✨ 📚Your perks for contribution to this community 👇🏻
Thanks again for your contribution! 😊 |
|
✅ Synchronized metadata from Issue #2071:
|
There was a problem hiding this comment.
Copilot review overview
🟡 Changes recommended
The new tooltip styling relies on clip-path without a robust cross-browser fallback and includes a CSS selector that is overly broad for the new structured tooltip content.
Get a fresh assessment by requesting another Copilot review.
Review effort: Lite
Findings: 1
Open (2)
What changed in this PR
Updates the LeaderBoard “Your badges” hover tooltip UI to better match the badge visuals by making the tooltip hexagonal and enriching its content.
Changes:
- Restructures the badge tooltip markup to include the badge image alongside the badge name.
- Updates CSS to remove circular cropping on badge icons and applies a hexagonal
clip-pathto the tooltip. - Adjusts tooltip layout to a flex row with spacing/padding to accommodate the new content.
| File | Description |
|---|---|
| src/components/dashboard/LeaderBoard/leaderboard.tsx | Updates tooltip markup to include the badge image + label text. |
| src/components/dashboard/LeaderBoard/leaderboard.css | Removes circular icon cropping and makes the tooltip hex-shaped with updated layout styles. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
| padding: 8px 12px; | ||
| display: flex; | ||
| align-items: center; | ||
| gap: 6px; | ||
| white-space: nowrap; |
| .sidebar-badge-tooltip img { | ||
| width: 42px; | ||
| height: 42px; | ||
| object-fit: contain; | ||
| } |


Description
Tried fixing hover ui/ux issue of leaderboard badges
Fixes #2071
Type of Change
Checklist
npm run buildand attached screenshot(s) in this PR.Was not able to test changes locally as Clerk is not configured locally, causing the leaderboard page to crash