fix(ui): remove redundant calendar icons from date inputs

[marlonwq]

🧭 Context

Resolves#2038

📚 Description

This PR removes the manual calendar icons, preventing duplication.

🛠️ Changes

• Removed redundant <span> icons with class i-lucide:calendar.
• Adjusted InputBase padding from ps-7 to ps-2 to align text correctly after icon removal.

🧪 Testing

• Verified on Chrome (Desktop): Only the native icon remains, picker works as expected.

[vercel]

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

Project	Deployment	Actions	Updated (UTC)
npmx.dev	Ready	Preview, Comment	Mar 12, 2026 8:34am

2 Skipped Deployments

Project	Deployment	Actions	Updated (UTC)
docs.npmx.dev	Ignored	Preview	Mar 12, 2026 8:34am
npmx-lunaria	Ignored		Mar 12, 2026 8:34am

[coderabbitai]

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: 74ac2bab-1283-4719-a687-fffc5db447c1

📥 Commits

Reviewing files that changed from the base of the PR and between 63f97d9 and 88be4a4.

📒 Files selected for processing (1)

• app/components/Package/TrendsChart.vue

🚧 Files skipped from review as they are similar to previous changes (1)

• app/components/Package/TrendsChart.vue

---

📝 Walkthrough

Walkthrough

Removed the calendar icon span elements that preceded the startDate and endDate inputs in TrendsChart.vue. Removed the ps-7 right-padding class from both date input elements, reducing horizontal spacing. No exported or public declarations were changed.

Possibly related PRs

• fix: prevent Chrome Android date input value clipping #2037: Adjusts date-input presentation in the same TrendsChart.vue component with WebKit/date-value styling tweaks.
• fix: improve versions modal input alignment #1553: Modifies spacing and icon/element markup for date inputs and related buttons in TrendsChart.vue.
• fix(ui): fix missing icons #1484: Changes icon-related markup in TrendsChart.vue, affecting calendar/icon rendering and formatting.

Suggested reviewers

• alexdln
• graphieros
• danielroe

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description check	✅ Passed	The PR description clearly relates to the changeset, describing removal of calendar icons and padding adjustment from TrendsChart.vue.
Linked Issues check	✅ Passed	The PR successfully addresses issue #2038 by removing redundant calendar icon spans and adjusting padding, meeting all stated coding requirements.
Out of Scope Changes check	✅ Passed	All changes in TrendsChart.vue are directly scoped to removing manual calendar icons and adjusting padding, with no 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

---

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

• X
• Mastodon
• Reddit
• LinkedIn

_{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!

[graphieros]

Thank you :)

[shuuji3]

LGTM👍🏻 I also tested on Firefox on desktop and Android.

The mobile view still hide the day, but I think this is a separate issue 😅

[alexdln]

@shuuji3 sorry, minor changes request 🙃

[shuuji3]

No problem! I thought I canceled the queue by approving it 😄

[graphieros]

The mobile view still hide the day, but I think this is a separate issue 😅

FIxed in https://github.com/npmx-dev/npmx.dev/pull/2037/changes

[marlonwq]

Wow, thanks for the quick reply!

I've implemented the suggestions 🙂

[graphieros]

I'm glad this one is in :)

[lukewarlow]

@graphieros that's not correct btw, firefox android does still have a bug but it seems related to the w-full and min-w-0 on the input. Chrome android had a separate bug that has been fixed by that patch.