Conversation
jagdish-15
left a comment
There was a problem hiding this comment.
The current hover effect looks good and fits well with the UX
But I think there’s been a slight misunderstanding of the issue. We wanted to apply the link to the usernames displayed on the leaderboard. That would make more sense than just adding the link behaviour to the names themselves.
Also, we should have some visual cue to indicate it’s clickable (even before hover), instead of relying only on the hover effect.
Got it!. I'm updating the PR to link the usernames and improve the visual indicators now. |
|
@avatGit What I meant earlier was that the hover effect was already good and fit the UX well. You only needed to switch the link from the Also, the recent commits seem to have removed some important parts of the main CSS, including the complete footer styles. I’d suggest doing. Let me know if anything’s still unclear |
f4eb304 to
2a80e10
Compare
|
@jagdish-15 My apologies for the previous mess. I've performed a clean reset to restore the footer and properly implemented the link on the username with the requested visual cue. Review please. Thanks again for your patience! |
Closes #15