Skip to content

Fix: Truncated Name in big repos#486

Merged
NorbertLoh merged 2 commits intoCATcher-org:mainfrom
lrongyi:username-tooltip
Jul 3, 2025
Merged

Fix: Truncated Name in big repos#486
NorbertLoh merged 2 commits intoCATcher-org:mainfrom
lrongyi:username-tooltip

Conversation

@lrongyi
Copy link
Copy Markdown
Contributor

@lrongyi lrongyi commented Jun 23, 2025

Summary:

Fixes #481

Type of change:

  • ✨ New Feature/ Enhancement

Changes Made:

  • Hovering over username will show a tooltip such that the user can view the full GitHub username, without needing to go to GitHub

Screenshots:

image

Proposed Commit Message:

Add tooltip to show full GitHub username on hover in card view

Long GitHub usernames are truncated with ellipses in the card view. 

This change adds a tooltip that displays the full username when 
hovered over, improving usability and clarity.

Checklist:

  • I have tested my changes thoroughly.
  • I have created tests for any new code files created in this PR or provided a link to a issue/PR that addresses this.
  • I have added or modified code comments to improve code readability where necessary.
  • I have updated the project's documentation as necessary.

Copy link
Copy Markdown
Contributor

@NorbertLoh NorbertLoh left a comment

Choose a reason for hiding this comment

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

Personally, I prefer just the name but if anyone prefers with the 'username: ' in #488. Please let me know. @lrongyi good job on the clear on concise PR. However, perhaps a more detailed git commit message will be better. You can see the format to follow at se-edu 'Commit message: Body' or previous PRs. Otherwise LGTM!

@lrongyi lrongyi requested a review from NorbertLoh June 30, 2025 02:53
Copy link
Copy Markdown
Contributor

@NorbertLoh NorbertLoh left a comment

Choose a reason for hiding this comment

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

LGTM! Please pull from the latest branch, and I will merge it. Thanks 👍

@lrongyi lrongyi requested a review from NorbertLoh July 1, 2025 14:39
@NorbertLoh NorbertLoh merged commit 3097053 into CATcher-org:main Jul 3, 2025
3 checks passed
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.

Truncated Name in big repos

2 participants