Skip to content

Score Breakdown Popup on Hover for Each User#40

Merged
jagdish-15 merged 9 commits into
codepvg:mainfrom
NandiniJeedimalla-07:score-tooltip-feature
May 31, 2026
Merged

Score Breakdown Popup on Hover for Each User#40
jagdish-15 merged 9 commits into
codepvg:mainfrom
NandiniJeedimalla-07:score-tooltip-feature

Conversation

@NandiniJeedimalla-07
Copy link
Copy Markdown
Contributor

What was added

Added a score tooltip feature to improve user understanding of score values in the score/leaderboard component.

Purpose

The tooltip provides additional contextual information when users hover over or interact with score elements, making the UI more informative and user-friendly.

Changes made

  • Implemented tooltip UI for score display
  • Added hover-based interaction for desktop users
  • Ensured responsive behavior across different screen sizes
  • Integrated tooltip with existing score component

Impact

Improves user experience by making score information clearer, more interactive, and easier to understand.

Screenshots

Attached screenshots show the tooltip behavior in both default and hover states.

  • Before feature
Screenshot 2026-05-29 132106
  • After feature
Screenshot 2026-05-29 132715

@NandiniJeedimalla-07
Copy link
Copy Markdown
Contributor Author

Resolves #20

@jagdish-15
Copy link
Copy Markdown
Collaborator

jagdish-15 commented May 29, 2026

Thanks for the implementation! A few things I'd like adjusted before merging:

  • The hover effect currently seems to be attached to the entire score container rather than just the score text itself. Since the hover area is noticeably larger than the visible text, could you make the trigger area match the score text more closely?
image
  • On mobile screens, the score breakdown tooltip overflows outside the viewport (towards the right side). Please ensure it remains fully visible on smaller devices.
  • The tooltip text could be changed from white to a slightly muted/faint green to better match the rest of the UI.
  • The border radius feels a bit large. Could you knock it back a few pixels?
  • One more UX suggestion: currently the tooltip remains visible when moving the cursor from the score text onto the tooltip itself. Since the tooltip is purely informational and doesn't contain any interactive elements, I think it would be better if it disappeared as soon as the cursor leaves the score text.

@jagdish-15 jagdish-15 added type:feature New feature or request level:beginner Indicates the difficulty labels May 29, 2026
@NandiniJeedimalla-07
Copy link
Copy Markdown
Contributor Author

yeah sure! I will work on it

@NandiniJeedimalla-07
Copy link
Copy Markdown
Contributor Author

Resolves #20
Addressed all requested changes. Let me know if anything else is needed.

Copy link
Copy Markdown
Collaborator

@jagdish-15 jagdish-15 left a comment

Choose a reason for hiding this comment

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

One thing I noticed is that the tooltip still remains visible when moving the cursor from the score onto the tooltip itself. Since the tooltip doesn't contain any interactive elements, I'd still prefer it to disappear as soon as the cursor leaves the score text.

Also, the feature is currently quite difficult to discover unless a user accidentally hovers over a score. Do you have any ideas for a subtle visual cue that indicates additional information is available, such as a small info icon or something similar? This isn't a hard requirement from the original issue, but I'd be interested in hearing your thoughts.

@NandiniJeedimalla-07
Copy link
Copy Markdown
Contributor Author

One subtle idea could be to add a small downward caret next to each score value, for example:

245 ▾
180 ▾
162 ▾

Many websites use a caret/chevron to indicate that additional information is available on hover or interaction. Since the score column already has an info icon explaining the scoring formula, the caret would specifically suggest that each individual score has its own details available, without adding much visual clutter to the leaderboard.

@jagdish-15
Copy link
Copy Markdown
Collaborator

That sounds like a reasonable and clean approach!

Feel free to go ahead with the caret indicator. Also, please make sure the interaction works well on mobile devices.

Once that's in place, I think we'll be in a good position to merge.

@jagdish-15
Copy link
Copy Markdown
Collaborator

Thanks for working through all the feedback on this one!

I took the liberty of making one small tweak before merging and changed the caret indicator to a > symbol, as I felt it fit the terminal-style theme of the project a bit better. Other than that, the implementation looks great.

@jagdish-15 jagdish-15 requested a review from codepvg May 31, 2026 18:46
@jagdish-15 jagdish-15 merged commit 7b68dbf into codepvg:main May 31, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

gssoc:approved level:beginner Indicates the difficulty quality:clean type:feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants