Skip to content

Adjust desktop nav link hit area#13

Closed
dogussyaman wants to merge 1 commit intomainfrom
codex/refactor-desktop-nav-for-link-and-motion
Closed

Adjust desktop nav link hit area#13
dogussyaman wants to merge 1 commit intomainfrom
codex/refactor-desktop-nav-for-link-and-motion

Conversation

@dogussyaman
Copy link
Copy Markdown
Owner

Motivation

  • Make the padded area of each desktop nav item fully clickable by moving motion interactions onto the Link element so the hit area includes padding.
  • Preserve the existing hover/active pill measurement and animation behavior for visual consistency.

Description

  • Introduce MotionLink = motion(Link) and use it in place of the previous motion.div + nested Link so the motion props are applied directly to the anchor.
  • Move padding and cursor styles onto the Link (now MotionLink) so the full padded area becomes clickable while keeping the visual text styles.
  • Keep the pill positioning logic (useLayoutEffect, itemRefs, and pill motion.li) intact so hover/active pill animation continues to work as before.

Testing

  • Started the dev server with npm run dev and Next.js reported Ready successfully.
  • Attempted a Playwright screenshot to validate the change, but the browser process crashed with a SIGSEGV and the screenshot run failed.
  • No unit or integration tests were executed as part of this change.

Codex Task

@vercel
Copy link
Copy Markdown

vercel Bot commented Feb 7, 2026

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

Project Deployment Actions Updated (UTC)
code-crafters Ready Ready Preview, Comment Feb 7, 2026 9:44am

@dogussyaman dogussyaman deleted the codex/refactor-desktop-nav-for-link-and-motion branch February 16, 2026 07:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant