fix: handle middle mouse button on click menu item

[H4ad]

📄 Summary

Add support to open the new tab not only via CTRL but also via middle click of the button, like URL link.

---

✅ Changes

• Feature: Brief description
• Bug fix: Brief description

---

🧪 How to Test

1. Open the initial page
2. Middle click in any menu item

---

🔍 Related Issues

I found no related issue.

---

📸 Screenshots / Screen Recording (if applicable / mandatory for UI related changes)

No screenshots since this change via click.

---

📋 Checklist

• Dev Review
• Test cases added (Unit/ Integration / E2E)
• Manually tested the changes

---

👀 Notes for Reviewers

Inspired by https://www.linkedin.com/feed/update/urn:li:activity:7400183369948520448?commentUrn=urn%3Ali%3Acomment%3A%28activity%3A7400183369948520448%2C7400648742405554176%29&dashCommentUrn=urn%3Ali%3Afsd_comment%3A%287400648742405554176%2Curn%3Ali%3Aactivity%3A7400183369948520448%29

---

Note

Enable opening sidebar items in new tabs via middle-click or Ctrl/Meta, with centralized event check and NavItem middle-button handling.

• Frontend – SideNav:
  • Introduce shouldMouseEventOpenInNewTab to detect Ctrl/Meta or middle-click and use it in onClickGetStarted, onClickHandler, and handleMenuItemClick to open routes in a new tab.
  • Update hooks dependencies to include shouldMouseEventOpenInNewTab.
• Frontend – NavItem:
  • Add onMouseDown to trigger onClick on middle mouse button when not disabled.

^{Written by Cursor Bugbot for commit 634cb52. This will update automatically on new commits. Configure here.}

[CLAassistant]

All committers have signed the CLA.

[cursor]

This PR is being reviewed by Cursor Bugbot

Details

Your team is on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle for each member of your team.

To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

[cursor]

Bug: Middle click on pin icons triggers unintended navigation

The new onMouseDown handler catches middle clicks on the entire nav-item div, including the Pin and PinOff icons inside it. The pin icons only have onClick handlers with stopPropagation(), which doesn't prevent mousedown events from bubbling. When a user middle-clicks on the pin icon, the mousedown event bubbles up to the parent div and triggers navigation (opening in a new tab), which is unintended behavior. The handler needs to check if the click target is the pin icon and skip navigation in that case.