fix: remove duplicate css classes

[TheAlexLichter]

All removed CSS classes are already part of the underlying components, causing HTML validator issues due to being duplicates when rendered.

[vercel]

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

Project	Deployment	Actions	Updated (UTC)
npmx.dev	Ready	Preview, Comment	Feb 11, 2026 11:14pm

2 Skipped Deployments

Project	Deployment	Actions	Updated (UTC)
docs.npmx.dev	Ignored	Preview	Feb 11, 2026 11:14pm
npmx-lunaria	Ignored		Feb 11, 2026 11:14pm

[coderabbitai]

📝 Walkthrough

Walkthrough

This pull request makes styling-only changes across four Vue components: it removes the font-mono utility from the maintainer name link in Maintainers.vue, removes a hover:text-fg hover class from a button in ManagerSelect.vue, simplifies ButtonBase classes (removing inline-flex, justify-center, duration-200) in Versions.vue, and simplifies button classes and removes a selected-state border in Compare/FacetSelector.vue. No logic, data flow, or public API signatures were changed.

Possibly related PRs

• refactor: remove unnecessary prop from links #1247: Overlapping refactor that removes font-mono/inline-flex and simplifies button/link classes across multiple components.
• refactor: replace links and buttons for compare and code page #1243: Direct edits to Compare/FacetSelector.vue that adjust button styling and selected-state classes.
• fix(i18n): various rtl and arabic fixes #760: Prior styling changes in Package components including Maintainers.vue and Versions.vue.

Suggested labels

front

Suggested reviewers

• alexdln
• danielroe

🚥 Pre-merge checks | ✅ 1

✅ Passed checks (1 passed)

Check name	Status	Explanation
Description check	✅ Passed	The pull request description clearly explains that removed CSS classes are duplicates already present in underlying components, directly relating to the changeset across all four Vue files.

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

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]

Actionable comments posted: 1

[coderabbitai]

⚠️ Potential issue | 🟡 Minor

Remove focus-visible:outline-accent/70 — focus styling is applied globally for buttons.

The inline focus-visible:outline-accent/70 utility class contradicts the project's convention. Focus-visible styling for <button> elements is already handled globally via main.css, so this class is unnecessary and may cause inconsistent focus states.

🔧 Suggested fix

-          class="gap-1 px-1.5 rounded transition-colors focus-visible:outline-accent/70"
+          class="gap-1 px-1.5 rounded transition-colors"

Based on learnings: "In the npmx.dev project, focus-visible styling for buttons and selects is applied globally via main.css... individual buttons or selects in Vue components should not rely on inline focus-visible utility classes like focus-visible:outline-accent/70."

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change

	class="gap-1 px-1.5 rounded transition-colors focus-visible:outline-accent/70"
	class="gap-1 px-1.5 rounded transition-colors"

[serhalp]

Thanks! I've been seeing these warnings locally too :)