Skip to content

Add tooltip to display full github username for big repos#488

Closed
SAN-MUYUN wants to merge 2 commits intoCATcher-org:mainfrom
SAN-MUYUN:enhancement/display-full-username
Closed

Add tooltip to display full github username for big repos#488
SAN-MUYUN wants to merge 2 commits intoCATcher-org:mainfrom
SAN-MUYUN:enhancement/display-full-username

Conversation

@SAN-MUYUN
Copy link
Copy Markdown
Contributor

@SAN-MUYUN SAN-MUYUN commented Jun 25, 2025

Summary:

Fixes #481

Type of change:

  • ✨ New Feature/ Enhancement

Changes Made:

Before:

Users are not able to see the full github username of assignee from WATcher application.

After:

By hovering over the truncated username on the card header, users can now see the full github username of the corresponding assignee.

Screenshots:

On mouse hover, the full username of the corresponding user will be displayed via the tooltip.
image

Proposed Commit Message:

Add tooltip to display full github username for big repos

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.

@NorbertLoh
Copy link
Copy Markdown
Contributor

It seems like there's a duplicate PR #488. However, you asked about it in #481. Perhaps you can discuss with each other. @SAN-MUYUN @lrongyi

@SAN-MUYUN
Copy link
Copy Markdown
Contributor Author

@lrongyi Sorry that I didn't see your PR earlier on as I had already asked to work on this issue.
Our codes look very similar with just a minor difference in the getAssigneeTooltip method, will it be fine to let @NorbertLoh decide whose PR to be merged?

@lrongyi
Copy link
Copy Markdown
Contributor

lrongyi commented Jun 27, 2025

@SAN-MUYUN My bad as well, I should have double checked before I made the PR.
I'm fine with letting @NorbertLoh decide too.

@NorbertLoh NorbertLoh mentioned this pull request Jun 29, 2025
4 tasks
@damithc
Copy link
Copy Markdown
Contributor

damithc commented Aug 6, 2025

@SAN-MUYUN Can you resolve the conflicts?

@SAN-MUYUN
Copy link
Copy Markdown
Contributor Author

SAN-MUYUN commented Aug 7, 2025

@damithc Hi prof, I have resolved the merge conflicts. However, this PR is very similar to a duplicated PR #486 which is already merged. Will it be fine for me to close this PR instead?

@damithc
Copy link
Copy Markdown
Contributor

damithc commented Aug 7, 2025

@damithc Hi prof, I have resolved the merge conflicts. However, this PR is very similar to a duplicated PR #486 which is already merged. Will it be fine for me to close this PR instead?

Sure @SAN-MUYUN, let's close this then.

@damithc damithc closed this Aug 7, 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.

Truncated Name in big repos

4 participants