Skip to content

Conversation

@piotr-bor
Copy link
Contributor

This commit introduces comprehensive improvements to navigation structure, dropdown menu functionality, and customer journey organization.

Navigation Component (Navigation.astro)

CSS Fixes for Dropdown Menu:

  • Fixed dropdown menu width issue by removing min-width: 280px and adding min-width: fit-content
  • Removed all unnecessary !important declarations (was causing CSS specificity issues)
  • Changed dropdown links from display: block to display: flex with proper alignment
  • Added width: 100% and justify-content: flex-start to ensure consistent left alignment
  • Added gap: 6px between text and external link icons
  • Fixed 'hover bridge' by changing pointer-events from none to auto in ::after pseudo-element
  • Ensured white-space: nowrap on both links and spans to prevent text wrapping

Result: Dropdown menus now properly size to content, text doesn't overflow background, and hover state is maintained when moving mouse from menu item to dropdown.

Navigation Data (nav.json)

Fixed Incorrect URLs:

  • Server: /product/server → /server
  • Client: /product/client → /client
  • Compare vs. Fortinet: /compare/defguard-vs-fortinet → /defguard-vs-fortinet
  • True VPN MFA: https://defguard.net/vpn_mfa/ → /vpn_mfa

Changed External Links to Internal:

Added Developer/Community Links to Resources:

Added Enterprise Features to Solutions:

Customer Journey Optimization

Removed FloatingLinks.astro:

  • Deleted redundant floating sidebar menu (created 'CTA soup')
  • Removed import and usage from BaseLayout.astro
  • Mixed personas (Adam/developers + Eva/business) in wrong context

Reorganized CTAs by Persona:

  • Eva (Business): Book a Demo remains prominent in main navigation
  • Adam (Developers): GitHub & Matrix moved to Resources dropdown
  • Newsletter: Moved from floating menu to footer (more natural placement)

Footer Redesign (Footer.astro)

Grid-Based Layout:

  • Replaced separate .footer-cols + .newsletter-section with unified .footer-grid
  • Desktop (lg): 4-column grid (1.5fr 1fr 1fr 1.5fr) - Logo | Company | Community | Newsletter
  • Tablet (md): 2-column grid
  • Mobile: Single column stack

Newsletter as Equal Column:

  • Positioned as last column in grid (equal importance)
  • Removed max-width, border-top/bottom, and padding constraints
  • Form now displays as vertical flex column (better UX in column context)
  • Full-width input and button within column

Strategic Benefits:

  • Newsletter treated as equal business goal, not an afterthought
  • Natural eye scanning across columns lands on conversion action
  • Professional SaaS-like appearance
  • Better mobile responsiveness

Impact

Before:

  • Broken dropdown menus (text overflow, couldn't click links)
  • Incorrect URLs leading to 404s
  • Redundant CTAs (Book a Demo in nav + floating menu)
  • Mixed audience targeting (developers + business in same widget)
  • Newsletter hidden at bottom

After:

  • ✅ Functional, properly-sized dropdown menus
  • ✅ All URLs corrected and working
  • ✅ Clean, persona-focused CTAs
  • ✅ Eva (business) → Book a Demo in nav
  • ✅ Adam (developers) → GitHub/Matrix in Resources
  • ✅ Newsletter prominent in footer grid
  • ✅ Professional, organized customer journey

This commit introduces comprehensive improvements to navigation structure,
dropdown menu functionality, and customer journey organization.

### Navigation Component (Navigation.astro)

**CSS Fixes for Dropdown Menu:**
- Fixed dropdown menu width issue by removing min-width: 280px and adding min-width: fit-content
- Removed all unnecessary !important declarations (was causing CSS specificity issues)
- Changed dropdown links from display: block to display: flex with proper alignment
- Added width: 100% and justify-content: flex-start to ensure consistent left alignment
- Added gap: 6px between text and external link icons
- Fixed 'hover bridge' by changing pointer-events from none to auto in ::after pseudo-element
- Ensured white-space: nowrap on both links and spans to prevent text wrapping

**Result:** Dropdown menus now properly size to content, text doesn't overflow background,
and hover state is maintained when moving mouse from menu item to dropdown.

### Navigation Data (nav.json)

**Fixed Incorrect URLs:**
- Server: /product/server → /server
- Client: /product/client → /client
- Compare vs. Fortinet: /compare/defguard-vs-fortinet → /defguard-vs-fortinet
- True VPN MFA: https://defguard.net/vpn_mfa/ → /vpn_mfa

**Changed External Links to Internal:**
- Prusa case study: https://defguard.net/blog/... → /blog/prusa-vpn-scaling-with-defguard
- Acquinox case study: https://defguard.net/blog/... → /blog/self-hosted-vpn-private-cloud-acquinox-defguard
(These now open in same tab instead of new tab)

**Added Developer/Community Links to Resources:**
- GitHub (Source) → https://github.com/DefGuard/defguard
- Matrix (Community) → https://matrix.to/#/#defguard:teonite.com
(Automatic external link detection via startsWith('http') opens in new tab)

**Added Enterprise Features to Solutions:**
- Enterprise Features → https://docs.defguard.net/enterprise/license
(Better placement than Product - it's a business solution, not a product feature)

### Customer Journey Optimization

**Removed FloatingLinks.astro:**
- Deleted redundant floating sidebar menu (created 'CTA soup')
- Removed import and usage from BaseLayout.astro
- Mixed personas (Adam/developers + Eva/business) in wrong context

**Reorganized CTAs by Persona:**
- Eva (Business): Book a Demo remains prominent in main navigation
- Adam (Developers): GitHub & Matrix moved to Resources dropdown
- Newsletter: Moved from floating menu to footer (more natural placement)

### Footer Redesign (Footer.astro)

**Grid-Based Layout:**
- Replaced separate .footer-cols + .newsletter-section with unified .footer-grid
- Desktop (lg): 4-column grid (1.5fr 1fr 1fr 1.5fr) - Logo | Company | Community | Newsletter
- Tablet (md): 2-column grid
- Mobile: Single column stack

**Newsletter as Equal Column:**
- Positioned as last column in grid (equal importance)
- Removed max-width, border-top/bottom, and padding constraints
- Form now displays as vertical flex column (better UX in column context)
- Full-width input and button within column

**Strategic Benefits:**
- Newsletter treated as equal business goal, not an afterthought
- Natural eye scanning across columns lands on conversion action
- Professional SaaS-like appearance
- Better mobile responsiveness

### Impact

**Before:**
- Broken dropdown menus (text overflow, couldn't click links)
- Incorrect URLs leading to 404s
- Redundant CTAs (Book a Demo in nav + floating menu)
- Mixed audience targeting (developers + business in same widget)
- Newsletter hidden at bottom

**After:**
- ✅ Functional, properly-sized dropdown menus
- ✅ All URLs corrected and working
- ✅ Clean, persona-focused CTAs
- ✅ Eva (business) → Book a Demo in nav
- ✅ Adam (developers) → GitHub/Matrix in Resources
- ✅ Newsletter prominent in footer grid
- ✅ Professional, organized customer journey
@mike767 mike767 merged commit 6eb4703 into main Oct 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants