Skip to content

fix: remove hidden overflow from sparkline wrapper - #1926

Merged
graphieros merged 1 commit into
npmx-dev:mainfrom
graphieros:main
Mar 4, 2026
Merged

fix: remove hidden overflow from sparkline wrapper#1926
graphieros merged 1 commit into
npmx-dev:mainfrom
graphieros:main

Conversation

@graphieros

Copy link
Copy Markdown
Member

This removes the hidden overflow from the sparkline wrapper, which cropped the selector when positioned on the last datapoint.

BeforeAfter
imageimage

@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.dev (canary)ReadyReadyPreview, CommentMar 4, 2026 4:49pm
2 Skipped Deployments
ProjectDeploymentActionsUpdated (UTC)
docs.npmx.devIgnoredIgnoredPreviewMar 4, 2026 4:49pm
npmx-lunariaIgnoredIgnoredMar 4, 2026 4:49pm

Request Review

@coderabbitai

Copy link
Copy Markdown
Contributor

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 0e614292-7472-4904-8761-fe864e5737e5

📥 Commits

Reviewing files that changed from the base of the PR and between f3403eb and 6410260.

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

📝 Walkthrough

Walkthrough

This pull request modifies the WeeklyDownloadStats.vue component by removing the overflow-hidden CSS class from the wrapper div that contains the weekly downloads sparkline visualisation. The height and egg-pulse animation classes are preserved, and no changes are made to the component's exported interfaces or data flow logic. The modification consists of one line added and four lines removed.

Possibly related PRs

Suggested reviewers

  • danielroe
  • 43081j
🚥 Pre-merge checks | ✅ 1
✅ Passed checks (1 passed)
Check nameStatusExplanation
Description check✅ PassedThe pull request description clearly relates to the changeset, explaining the removal of overflow-hidden and providing before/after screenshots demonstrating the fix.

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

Tip

Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).
Share your feedback on Discord.


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.

@codecov

codecovBot commented Mar 4, 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!

@graphieros
graphieros enabled auto-merge March 4, 2026 16:51
@graphieros
graphieros added this pull request to the merge queueMar 4, 2026
Merged via the queue into npmx-dev:main with commit 462fb98Mar 4, 2026
19 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.

2 participants

@graphieros@ghostdevv