Skip to content

Conversation

@Biki-dev
Copy link
Contributor

@Biki-dev Biki-dev commented Nov 28, 2025

Summary

This PR fixes the stats cards UI by adding a centered animated hover tooltip that displays the full stat title and removes hard truncation. It also enables wrapping/line-clamp fallback so titles are readable across screen sizes without breaking layout.

Changes

  • Replaced manual substring truncation with line-clamp-2 / CSS fallback for multi-line wrapping.
  • Added a centered, animated Framer Motion tooltip that shows full title on hover.
  • Ensured accessible text with title and aria-label.
  • Added min-h to stat cards to keep visual alignment when titles wrap.
  • Committed formatting and lint fixes as needed.

Before & After Comparison

❌ Before (Issue)

Screenshot 2025-11-28 194027

✔ After (Improved Behavior)

Screenshot 2025-11-28 201809 Screenshot 2025-11-28 201800 Screenshot 2025-11-28 201752

QA checklist (done before PR)

  • npm run format completed
  • npm run lint:ts passed
  • npm run lint:md passed
  • npm run build passed

@github-actions
Copy link

🎉 All Checks Passed!

Status: ✅ Ready to merge

✅ Completed Workflows

Workflow Status Details
🔨 Continuous Integration ✅ Passed Build completed successfully
📝 Code Linting ✅ Passed All formatting and style checks passed

🚀 This PR is ready for review and can be safely merged to main branch!

Great work! Your code meets all quality standards. 👏

@Biki-dev
Copy link
Contributor Author

Hi @sa-fw-an — I fixed the stats card truncation and added a centered hover tooltip that shows full titles with animation.
Please review when you have a moment. Screenshots are attached.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please do not push the files that you have not edited. The PR is great, but I want you to push only the Stats.tsx file and not the package-lock.json

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hello, @sa-fw-an I updated the branch and restored package-lock.json to match main.
Only the Stats component changes are now included in the PR.

@github-actions
Copy link

🎉 All Checks Passed!

Status: ✅ Ready to merge

✅ Completed Workflows

Workflow Status Details
🔨 Continuous Integration ✅ Passed Build completed successfully
📝 Code Linting ✅ Passed All formatting and style checks passed

🚀 This PR is ready for review and can be safely merged to main branch!

Great work! Your code meets all quality standards. 👏

@sa-fw-an sa-fw-an closed this Nov 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants