Skip to content

Comments

fix: add missing padding#1589

Merged
danielroe merged 1 commit intonpmx-dev:mainfrom
graphieros:main
Feb 22, 2026
Merged

fix: add missing padding#1589
danielroe merged 1 commit intonpmx-dev:mainfrom
graphieros:main

Conversation

@graphieros
Copy link
Contributor

Minor polishing, adds padding on 2 elements of the side panel to fix the cropping of the focus rings:

  • sparkline animation toggle button
  • versions filter input

@vercel
Copy link

vercel bot commented Feb 22, 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 22, 2026 9:57pm
2 Skipped Deployments
Project Deployment Actions Updated (UTC)
docs.npmx.dev Ignored Ignored Preview Feb 22, 2026 9:57pm
npmx-lunaria Ignored Ignored Feb 22, 2026 9:57pm

Request Review

@codecov
Copy link

codecov bot commented Feb 22, 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!

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 22, 2026

No actionable comments were generated in the recent review. 🎉


📝 Walkthrough

Walkthrough

Two Vue components in the package display interface received minor UI padding adjustments. The Versions component gained additional vertical padding (py-1) around its semver filter input container, whilst the WeeklyDownloadStats component received general padding (p-1) to its wrapper div. No functional logic, data flow, or event handling was modified in either component.

Possibly related PRs

Suggested labels

front, a11y

Suggested reviewers

  • danielroe
🚥 Pre-merge checks | ✅ 1
✅ Passed checks (1 passed)
Check name Status Explanation
Description check ✅ Passed The pull request description clearly describes the changes made: adding padding to two elements (sparkline animation toggle button and versions filter input) to fix focus ring cropping.

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

@danielroe danielroe added this pull request to the merge queue Feb 22, 2026
Merged via the queue into npmx-dev:main with commit 6d521a0 Feb 22, 2026
16 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.

2 participants