Skip to content

fix(header): make Education and Research nav parents clickable#123

Merged
digisus merged 1 commit into
mainfrom
fix/clickable-nav-parents
May 6, 2026
Merged

fix(header): make Education and Research nav parents clickable#123
digisus merged 1 commit into
mainfrom
fix/clickable-nav-parents

Conversation

@stefan-briggs

Copy link
Copy Markdown
Contributor

Summary

  • Education and Research nav parents are now clickable links to their landing pages
  • Dropdowns open on hover/focus instead of click-only — the whole parent label is one click target
  • About is unchanged for now (no landing page yet)

Closes #122

Test plan

  • Hover "Education" in header → dropdown opens
  • Click "Education" → navigates to /education
  • Click a sub-item (e.g. "Private Bitcoin Briefing") → navigates correctly
  • Same checks for Research
  • Hover About → dropdown still opens; clicking About does not navigate (intended)
  • Mobile (narrow viewport): tap Education/Research label → navigates; About label stays as plain text with sub-items listed below

…122)

The dropdowns were the only interactive element on these parents, so the
/education and /research landing pages were unreachable from the header.
Render the parents as Links with hover/focus opening the dropdown so the
whole label is one click target. About is unchanged since it has no
landing page yet.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@stefan-briggs
stefan-briggs requested a review from digisus May 6, 2026 12:18
@github-actions

github-actions Bot commented May 6, 2026

Copy link
Copy Markdown
Contributor

Vercel Preview Deployment

✅ Preview ready for commit d8010e9

🔗 Preview URL: https://sbi-website-68hyin97i-digital-trust-solutions-gmb-h.vercel.app

@digisus

digisus commented May 6, 2026

Copy link
Copy Markdown
Contributor

All good according to spec.
Just, I hope people realize that they CAN click on education/research... Not so sure after trying it out, tbh.
Is a questio for a UX expert. But definitely better now than before, so I tend to merge the PR.

@digisus
digisus merged commit 6830593 into main May 6, 2026
1 check passed
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.

Stefan + Claude First Website Change + Fixing Small Bug

2 participants