Skip to content

fix(ui): dependency name readability - #1933

Merged
ghostdevv merged 3 commits into
npmx-dev:mainfrom
MatteoGabriele:fix/dependency-name-readability
Mar 4, 2026
Merged

fix(ui): dependency name readability#1933
ghostdevv merged 3 commits into
npmx-dev:mainfrom
MatteoGabriele:fix/dependency-name-readability

Conversation

@MatteoGabriele

@MatteoGabrieleMatteoGabriele commented Mar 4, 2026

Copy link
Copy Markdown
Member

🔗 Linked issue

closes#1925

🧭 Context

The names of dependencies become unreadable when they are very long.

The dependencies simply wrap so they remain readable even when they're long.

📚 Description

The dependency names are hard to read, and I don't think adding tooltips is the best fix. Since this is a key functional part of the package view, the names should be easily visible.

before
Image

after
Image

@vercel

vercelBot commented Mar 4, 2026

Copy link
Copy Markdown
Contributor

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

ProjectDeploymentActionsUpdated (UTC)
npmx.devReadyReadyPreview, CommentMar 4, 2026 10:26pm
2 Skipped Deployments
ProjectDeploymentActionsUpdated (UTC)
docs.npmx.devIgnoredIgnoredPreviewMar 4, 2026 10:26pm
npmx-lunariaIgnoredIgnoredMar 4, 2026 10:26pm

Request Review

@coderabbitai

coderabbitaiBot commented Mar 4, 2026

Copy link
Copy Markdown
Contributor
📝 Walkthrough

Walkthrough

This change updates app/components/Package/Dependencies.vue to adjust layout and truncation for dependency lists: increase gaps between name and version, set dependency name containers to larger max widths with word-wrap (e.g. max-w-[70%]/max-w-[80%] depending on section), reduce version link max width from 40% to 30%, and change optional-dependencies row alignment from items-center to items-baseline. No changes to data fetching, conditional rendering, or exported/public API declarations.

Possibly related PRs

Suggested reviewers

  • danielroe
  • 43081j
🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check nameStatusExplanation
Linked Issues check✅ PassedThe PR successfully addresses issue #1925 by implementing layout changes that allow dependency names to wrap and remain readable instead of being truncated.
Out of Scope Changes check✅ PassedAll changes are scoped to the Dependencies.vue component and focus on improving readability of dependency names through CSS adjustments, directly related to the linked issue.
Description check✅ PassedThe pull request description clearly relates to the changeset, explaining the problem (unreadable long dependency names) and solution (wrapping names).

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

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.

@coderabbitaicoderabbitaiBot left a comment

Copy link
Copy Markdown
Contributor

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
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 96973b2e-8c09-4dbe-8335-e3340fed0e48

📥 Commits

Reviewing files that changed from the base of the PR and between 11c2116 and 4cc7485.

📒 Files selected for processing (1)
  • app/components/Package/Dependencies.vue

Comment threadapp/components/Package/Dependencies.vue
@codecov

codecovBot commented Mar 4, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 50.00000% with 1 line in your changes missing coverage. Please review.
✅ All tests successful. No failed tests found.

Files with missing linesPatch %Lines
app/components/Package/Dependencies.vue50.00%1 Missing ⚠️

📢 Thoughts on this report? Let us know!

@ghostdevv
ghostdevv added this pull request to the merge queueMar 4, 2026
Merged via the queue into npmx-dev:main with commit 51ecbc7Mar 4, 2026
21 checks passed
@github-actionsgithub-actionsBot mentioned this pull request Mar 4, 2026
Sign up for freeto 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.

dependency names are missing a tooltip and versions are cut off

2 participants

@MatteoGabriele@ghostdevv