Skip to content

Comments

chore: add minor effect for home search input#1567

Draft
alexdln wants to merge 3 commits intonpmx-dev:mainfrom
alexdln:chore/home-input-effect
Draft

chore: add minor effect for home search input#1567
alexdln wants to merge 3 commits intonpmx-dev:mainfrom
alexdln:chore/home-input-effect

Conversation

@alexdln
Copy link
Member

@alexdln alexdln commented Feb 22, 2026

🧭 Context

Experimenting

📚 Description

@vercel
Copy link

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

Request Review

@codecov
Copy link

codecov bot commented Feb 22, 2026

Codecov Report

❌ Patch coverage is 0% with 12 lines in your changes missing coverage. Please review.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
app/pages/index.vue 0.00% 8 Missing and 4 partials ⚠️

📢 Thoughts on this report? Let us know!

Comment on lines +29 to +37
function onMouseMove(event: MouseEvent) {
const inputWrapper = inputWrapperRef.value
if (!inputWrapper) return
const rect = inputWrapper.getBoundingClientRect()
const x = event.clientX - rect.left
const y = event.clientY - rect.top
inputWrapper.style.setProperty('--mouse-x', `${x}px`)
inputWrapper.style.setProperty('--mouse-y', `${y}px`)
}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

And I even touched this somewhere. Thanks for the tip ❤️

Let's see if we need this effect and if so I'll update it then

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.

2 participants