fix: Mobile navigation issue on blog pages

[canerakdas]

Description

Fixes the mobile navigation issue on blog pages

Validation

When a user selects a category from the dropdown on the blog pages, they should be redirected to the corresponding category page

Related Issues

Fixes #8296

[vercel]

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

Project	Deployment	Preview	Updated (UTC)
nodejs-org	Ready	Preview	Oct 31, 2025 4:52pm

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 76.60%. Comparing base (a2bd7da) to head (9e1af7d).
⚠️ Report is 1 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #8302   +/-   ##
=======================================
  Coverage   76.60%   76.60%           
=======================================
  Files         117      117           
  Lines        9746     9746           
  Branches      328      328           
=======================================
  Hits         7466     7466           
  Misses       2278     2278           
  Partials        2        2           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[Copilot]

Pull Request Overview

This PR adds support for custom link components in the StatelessSelect component used by BaseLinkTabs. The change enables the dropdown select to use the same custom link component (e.g., Next.js Link, React Router Link) as the tab buttons.

• Passes the as prop (containing the custom Component) to StatelessSelect

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[canerakdas]

With this PR, only fixed the redirection issue, but I don’t quite understand why we’re using StatelessSelect directly. We should use WithNoScriptSelect and fall back to the StatelessSelect component only when JavaScript is disabled

StatelessSelect alone is unlikely to provide the accessibility benefits we achieve with Radix, so it should only be used as a last resort 🪳

[canerakdas]

Requesting fast-track, as this fixes an issue that was a blocker for mobile users. @nodejs/nodejs-website

[avivkeller]

With this PR, only fixed the redirection issue, but I don’t quite understand why we’re using StatelessSelect directly. We should use WithNoScriptSelect and fall back to the StatelessSelect component only when JavaScript is disabled

We are using it so we can render this component entirely on the server-side

[ovflowd]

With this PR, only fixed the redirection issue, but I don’t quite understand why we’re using StatelessSelect directly. We should use WithNoScriptSelect and fall back to the StatelessSelect component only when JavaScript is disabled

We are using it so we can render this component entirely on the server-side

But that select is still not compatible with JavaScript being disabled, no? So even being Stateless, does it work with JS disabled?

[avivkeller]

So even being Stateless, does it work with JS disabled?

Yes. JavaScript is not needed to use the StatelessSelect, as it uses the <details /> component