fix(nav): keep product topics reachable on narrow screens#787
Conversation
The secondary nav bar is hidden at max-width 60rem, but the sidebar-topics fallback only reappeared at max-width 48rem, leaving a 48-60rem dead zone with no reachable top navigation. Align the fallback breakpoint to 60rem so topics show in the sidebar exactly when the bar is hidden.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (1)
📜 Recent review details⏰ Context from checks skipped due to timeout. (3)
🧰 Additional context used📓 Path-based instructions (4)**/*.{md,mdx,astro,ts}📄 CodeRabbit inference engine (CONTRIBUTING.md)
Files:
src/components/**/*.astro📄 CodeRabbit inference engine (CONTRIBUTING.md)
Files:
src/components/SecondaryNav.astro📄 CodeRabbit inference engine (CLAUDE.md)
Files:
**⚙️ CodeRabbit configuration file
Files:
🧠 Learnings (1)📚 Learning: 2026-02-25T12:04:06.383ZApplied to files:
🔇 Additional comments (1)
WalkthroughThe secondary navigation stylesheet updates the breakpoint that reveals the global sidebar topics list, changing it from the mobile threshold to ChangesSidebar topics visibility
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
✅ Deploy Preview for scalekit-starlight ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |

The secondary nav bar is hidden at max-width 60rem, but the sidebar-topics fallback only reappeared at max-width 48rem, leaving a 48-60rem dead zone with no reachable top navigation. Align the fallback breakpoint to 60rem so topics show in the sidebar exactly when the bar is hidden.
Summary by CodeRabbit