Skip to content

fix: align deprecated version icon and text - #1125

Merged
danielroe merged 1 commit into
npmx-dev:mainfrom
iiio2:fix/align-deprecated-version-icon-and-text
Feb 7, 2026
Merged

fix: align deprecated version icon and text#1125
danielroe merged 1 commit into
npmx-dev:mainfrom
iiio2:fix/align-deprecated-version-icon-and-text

Conversation

@iiio2

@iiio2iiio2 commented Feb 7, 2026

Copy link
Copy Markdown
Contributor

I think we can improve the alignment.

Before:
before

After:
after

@vercel

vercelBot commented Feb 7, 2026

Copy link
Copy Markdown
Contributor

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

ProjectDeploymentActionsUpdated (UTC)
npmx.devReadyReadyPreview, CommentFeb 7, 2026 5:09am
2 Skipped Deployments
ProjectDeploymentActionsUpdated (UTC)
docs.npmx.devIgnoredIgnoredPreviewFeb 7, 2026 5:09am
npmx-lunariaIgnoredIgnoredFeb 7, 2026 5:09am

Request Review

@codecov

codecovBot commented Feb 7, 2026

Copy link
Copy Markdown

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!

@coderabbitai

Copy link
Copy Markdown
Contributor
📝 Walkthrough

Walkthrough

This pull request modifies the app/components/Package/Versions.vue file, updating UI class attributes on multiple NuxtLink elements. The "block" display class has been removed from several elements, changing their rendering from block-level to inline-level layout. The changes affect 6 lines of code with no modifications to component logic, data flow, conditional behaviour, or public API signatures. All other styling and functionality remains intact.

Suggested reviewers

  • danielroe
🚥 Pre-merge checks | ✅ 1
✅ Passed checks (1 passed)
Check nameStatusExplanation
Description check✅ PassedThe pull request description is directly related to the changeset, providing visual before/after comparisons of the alignment improvements made to the deprecated version icon and text in the Versions.vue component.

✏️ 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

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

@danielroedanielroe left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

nice 👌

@danielroe
danielroe added this pull request to the merge queueFeb 7, 2026
Merged via the queue into npmx-dev:main with commit 1bf177bFeb 7, 2026
17 checks passed
@iiio2

iiio2 commented Feb 7, 2026

Copy link
Copy Markdown
ContributorAuthor

Thanks.

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.

2 participants

@iiio2@danielroe