fix: add cursor pointer to button and list items

[NandkishorJadoun]

Description

Previously, the trigger button and dropdown options showed the default arrow cursor, which made the UI feel non interactive. This pr updates the cursor style for the package manager selector to use a pointer cursor on interactive elements.

Changes:

• Added cursor-pointer to the dropdown trigger
• Added cursor-pointer to clickable dropdown options

[vercel]

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

Project	Deployment	Actions	Updated (UTC)
npmx.dev	Ready	Preview, Comment	Feb 10, 2026 6:53am

2 Skipped Deployments

Project	Deployment	Actions	Updated (UTC)
docs.npmx.dev	Ignored	Preview	Feb 10, 2026 6:53am
npmx-lunaria	Ignored		Feb 10, 2026 6:53am

[coderabbitai]

📝 Walkthrough

Walkthrough

The pull request modifies the ManagerSelect component to add cursor-pointer styling classes to interactive elements. Specifically, the cursor-pointer utility class is added to the trigger button and each option item within the component. The change consists of two lines added and two lines removed, affecting only the class bindings without altering control flow, event handling, data bindings, or accessibility properties.

Possibly related PRs

• feat: use default cursor for non-link items #1216: Modifies cursor classes on the same ManagerSelect trigger and option item elements, making opposing cursor-related changes to the same interactive components.

Suggested reviewers

• alexdln
• danielroe

🚥 Pre-merge checks | ✅ 1

✅ Passed checks (1 passed)

Check name	Status	Explanation
Description check	✅ Passed	The pull request description accurately describes the changeset, explaining the cursor style improvements to interactive elements in the package manager selector.

_{✏️ 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.}

[coderabbitai]

Actionable comments posted: 1

[coderabbitai]

⚠️ Potential issue | 🟡 Minor

Remove the per-element focus-visible utility from the button class.

Focus-visible styling for buttons is already handled globally; keeping focus-visible:outline-accent/70 here conflicts with the project rule. Please drop it while you’re touching this class list.

Proposed change

- class="cursor-pointer flex items-center gap-1.5 px-2 py-2 font-mono text-xs text-fg-muted bg-bg-subtle border border-border-subtle border-solid rounded-md transition-colors duration-150 hover:(text-fg border-border-hover) active:scale-95 focus:border-border-hover focus-visible:outline-accent/70 hover:text-fg"+ class="cursor-pointer flex items-center gap-1.5 px-2 py-2 font-mono text-xs text-fg-muted bg-bg-subtle border border-border-subtle border-solid rounded-md transition-colors duration-150 hover:(text-fg border-border-hover) active:scale-95 focus:border-border-hover hover:text-fg"

Based on learnings: “In the npmx.dev project, ensure that focus-visible styling for button and select elements is implemented globally in app/assets/main.css… Do not apply per-element inline utility classes like focus-visible:outline-accent/70 on these elements.”

[codecov]

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!