-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add navigation menu? #93
Comments
I like this idea (and I also tried this out locally) but I think we should settle on the Information architecture first? I'm not yet sure whether Templates should stay on the documentation site or be in the kit only or remain in both - see #60 Also had an idea that we could add Support (#91) and Contribution (#92) pages which could then go in the nav? Also do we want to stick with 'How to guides' or pick another label for that? 🤷 |
This adds a "primary nav" with 3 items: * Get started * Guides * Support "Contribute" could be added when #148 is merged. I’ve not included "Page templates" as that page feels less valuable longer-term (eg if the NHS website templates get moved to a plugin or something). Adding the primary nav possibly means that some of the breadcrumbs could be removed or shortened (eg the "Home" breadcrumb link on Support feels a bit odd now?), but the Service Manual website has both, so maybe it’s fine? There’s no "current section" indicator for now, but we can adopt that when it gets added to the header in the next major release of NHS frontend (hopefully). Part of #93
Closing this issue as I think it is covered by other thigns |
Using a structure like NHS notify would allow us to both have a consistent navigation and also free up the area underneath the hero image to have key benefits.
The text was updated successfully, but these errors were encountered: