Skip to content

Commit faaf023

Browse files
Sbussisoclaude
andcommitted
fix(nav): collapse marketing links when signed in to stop overflow
Adding /security and /pricing in 9ac7e8d pushed the signed-in nav past its max-width: logo + 6 marketing links + OrganizationSwitcher + 3 in-product links + UserButton stopped fitting in the 1200px container, so they collapsed into each other ("SentryFeatures" jammed at the logo, "PricingR Sbussiso..." jammed at the org switcher). Hide the marketing <ul> when signed in — the in-product nav already covers what a logged-in user needs. Surface a single Docs link inside SignedInActions so the only marketing page still useful post-sign-in stays one click away. Logged-out users see the full marketing nav as before. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
1 parent fafcf73 commit faaf023

1 file changed

Lines changed: 29 additions & 20 deletions

File tree

frontend/src/components/LandingNav.jsx

Lines changed: 29 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -33,6 +33,9 @@ function SignedInActions() {
3333
</Link>
3434
</>
3535
)}
36+
<Link to="/docs" className="nav-link">
37+
Docs
38+
</Link>
3639
</nav>
3740
</>
3841
)}
@@ -53,26 +56,32 @@ function LandingNav() {
5356
<span className="landing-logo-text">Sentry</span>
5457
</Link>
5558

56-
<ul className={`landing-nav-links ${mobileMenuOpen ? 'active' : ''}`}>
57-
<li>
58-
<Link to="/#features" onClick={() => setMobileMenuOpen(false)}>Features</Link>
59-
</li>
60-
<li>
61-
<Link to="/#architecture" onClick={() => setMobileMenuOpen(false)}>Architecture</Link>
62-
</li>
63-
<li>
64-
<Link to="/#quickstart" onClick={() => setMobileMenuOpen(false)}>Quick Start</Link>
65-
</li>
66-
<li>
67-
<Link to="/docs" onClick={() => setMobileMenuOpen(false)}>Docs</Link>
68-
</li>
69-
<li>
70-
<Link to="/security" onClick={() => setMobileMenuOpen(false)}>Security</Link>
71-
</li>
72-
<li>
73-
<Link to="/pricing" onClick={() => setMobileMenuOpen(false)}>Pricing</Link>
74-
</li>
75-
</ul>
59+
{/* Marketing nav. Hidden when signed in — the in-product nav
60+
(Dashboard / Settings / Admin / Docs from SignedInActions) takes
61+
over so the row doesn't overflow. Docs is the only marketing
62+
link still useful in-product, so it appears in both nav modes. */}
63+
<SignedOut>
64+
<ul className={`landing-nav-links ${mobileMenuOpen ? 'active' : ''}`}>
65+
<li>
66+
<Link to="/#features" onClick={() => setMobileMenuOpen(false)}>Features</Link>
67+
</li>
68+
<li>
69+
<Link to="/#architecture" onClick={() => setMobileMenuOpen(false)}>Architecture</Link>
70+
</li>
71+
<li>
72+
<Link to="/#quickstart" onClick={() => setMobileMenuOpen(false)}>Quick Start</Link>
73+
</li>
74+
<li>
75+
<Link to="/docs" onClick={() => setMobileMenuOpen(false)}>Docs</Link>
76+
</li>
77+
<li>
78+
<Link to="/security" onClick={() => setMobileMenuOpen(false)}>Security</Link>
79+
</li>
80+
<li>
81+
<Link to="/pricing" onClick={() => setMobileMenuOpen(false)}>Pricing</Link>
82+
</li>
83+
</ul>
84+
</SignedOut>
7685

7786
<div className="landing-nav-actions">
7887
<SignedOut>

0 commit comments

Comments
 (0)