Skip to content

fix: package sidebar glitching#947

Open
iiio2 wants to merge 1 commit intonpmx-dev:mainfrom
iiio2:fix/package-sidebar-glitching
Open

fix: package sidebar glitching#947
iiio2 wants to merge 1 commit intonpmx-dev:mainfrom
iiio2:fix/package-sidebar-glitching

Conversation

@iiio2
Copy link
Contributor

@iiio2 iiio2 commented Feb 4, 2026

resolves #937

@vercel
Copy link

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

Request Review

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 4, 2026

📝 Walkthrough

Walkthrough

Removed the hover:pe-0.5 class from the sidebar container in app/pages/package/[...package].vue, so hovering no longer alters end padding. Change is a single-line edit; no exported or public entities were modified.

Possibly related PRs

Suggested reviewers

  • danielroe
🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description check ✅ Passed The pull request description references issue #937 which directly relates to the code changes that fix the sidebar glitching problem.
Linked Issues check ✅ Passed The code removes hover-induced padding-right reduction from the sidebar container, which addresses the glitching issue related to scrollbar and padding interactions reported in issue #937.
Out of Scope Changes check ✅ Passed The change is narrowly focused on removing the problematic hover:pe-0.5 class from the sidebar container, directly addressing the reported glitching issue without introducing unrelated modifications.

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

@codecov
Copy link

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

@graphieros
Copy link
Contributor

It appears to still be glitching in preview

@iiio2
Copy link
Contributor Author

iiio2 commented Feb 5, 2026

I think the cause of glitching is hover:pe-0.5 class from the sidebar div. This was causing the sidebar to change its padding-end on hover (from pe-2.5 to pe-0.5), which likely caused the "glitching" effect. I have tested in desktop + mobile and now it works well.

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.

fix: package sidebar glitching

2 participants