Skip to content

fix(ui): truncate long package names on profile package likes page#1757

Merged
alexdln merged 2 commits intomainfrom
serhalp/fix-profile-pkg-overflow
Feb 28, 2026
Merged

fix(ui): truncate long package names on profile package likes page#1757
alexdln merged 2 commits intomainfrom
serhalp/fix-profile-pkg-overflow

Conversation

@serhalp
Copy link
Copy Markdown
Member

@serhalp serhalp commented Feb 28, 2026

🔗 Linked issue

N/A

🧭 Context

Very long package names overflow on the name package like page:

Screenshot 2026-02-28 at 10 37 08

📚 Description

Truncate the package name with pure CSS while keeping the full name in the title attribute so it can be seen on hover (and by screen readers):

Screenshot 2026-02-28 at 11 40 28

@vercel
Copy link
Copy Markdown

vercel Bot commented Feb 28, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
npmx.dev Ready Ready Preview, Comment Feb 28, 2026 4:49pm
2 Skipped Deployments
Project Deployment Actions Updated (UTC)
docs.npmx.dev Ignored Ignored Preview Feb 28, 2026 4:49pm
npmx-lunaria Ignored Ignored Feb 28, 2026 4:49pm

Request Review

@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 28, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

@serhalp serhalp marked this pull request as ready for review February 28, 2026 17:34
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Feb 28, 2026

📝 Walkthrough

Walkthrough

The LikeCard component has been updated to improve handling of long package names through UI adjustments including truncation and tooltip display. The package name is now rendered within a truncate-enabled span with a title attribute for hover information, whilst the action area is wrapped in a shrink-0 container. A new comprehensive test file has been added to verify the component's rendering, truncation behaviour, tooltip functionality, and extraction of scoped package names from URLs.

Suggested reviewers

  • danielroe
  • whitep4nth3r
🚥 Pre-merge checks | ✅ 1
✅ Passed checks (1 passed)
Check name Status Explanation
Description check ✅ Passed The pull request description clearly relates to the changeset, explaining the problem (long package names overflow) and the solution (CSS truncation with title attribute).

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch serhalp/fix-profile-pkg-overflow

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1


ℹ️ Review info

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between e5ccbff and 9fb88a4.

📒 Files selected for processing (2)
  • app/components/Package/LikeCard.vue
  • test/nuxt/components/PackageLikeCard.spec.ts

Comment thread test/nuxt/components/PackageLikeCard.spec.ts
@alexdln alexdln added this pull request to the merge queue Feb 28, 2026
Merged via the queue into main with commit 04bd490 Feb 28, 2026
21 checks passed
@alexdln alexdln deleted the serhalp/fix-profile-pkg-overflow branch February 28, 2026 17:55
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