Update menu behavior to click-to-open and fix hover styling

[itasimo]

Closes #127

[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: 41489b1e-41db-4b04-a678-9e1326526594

📥 Commits

Reviewing files that changed from the base of the PR and between 146617a and 263bfe5.

📒 Files selected for processing (1)

• src/components/header/desktop-layout.tsx

---

Walkthrough

DesktopLayout adds pointer event handlers to dropdown menu triggers that suppress default pointer behavior, and updates the hover effect class styling with trailing whitespace. These changes address dropdown menu interaction behavior in the desktop navigation layout.

Changes

Navigation Menu Trigger

Layer / File(s)	Summary
Dropdown menu trigger pointer behavior src/components/header/desktop-layout.tsx	NavigationMenuTrigger for dropdowns gains onPointerEnter, onPointerLeave, and onPointerMove handlers that call preventDefault() to suppress default pointer interaction. The customHoverEffectClass string is updated with trailing whitespace.

🚥 Pre-merge checks | ✅ 2 | ❌ 2

❌ Failed checks (2 inconclusive)

Check name	Status	Explanation	Resolution
Linked Issues check	❓ Inconclusive	The code changes attempt to address issue #127 by adding pointer event handlers to prevent default behavior, but incomplete implementation leaves uncertain whether click-to-open menu behavior and hover styling fixes are fully realized.	Verify that pointer event handlers properly enable click-to-open menu behavior, confirm hover styling fixes achieve intended effect, and ensure other files aren't required for complete functionality.
Out of Scope Changes check	❓ Inconclusive	The trailing whitespace addition to customHoverEffectClass appears incidental to the main objective of fixing menu behavior and hover styling.	Clarify whether the trailing whitespace in customHoverEffectClass is intentional and necessary for the hover styling fix, or if it should be removed.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately describes the main changes: updating menu behavior to click-to-open and fixing hover styling, which aligns with the code modifications to prevent default pointer behavior and adjust hover effect classes.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

---

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.}