Skip to content
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

[WIP] Multipage navigation #100

Closed
wants to merge 2 commits into from
Closed

[WIP] Multipage navigation #100

wants to merge 2 commits into from

Conversation

MatMoore
Copy link
Contributor

This branch is to try and get the navigation into a tree structure. The current live version is generated only from the index page, so most pages are not shown in the nav, and some pages are not even discoverable at the moment (like the resource lists)

Currently we are having problems with

  • Some navigation entries having invisible titles
  • It seems like everything shows up at the top level, instead of just the index, resources, and career path at the top level

screen shot 2018-08-24 at 16 41 34

However, we are missing the middleman metadata stuff at the top of each file, so that might be why it renders strangely. I'm also having trouble at the moment with middleman serve hanging which is stopping me from testing.

Changes on this branch

  • Enabled the multi page and collapsable nav options
  • Moved the "inspiration" page to the resources section since this shouldn't be a top level thing. This might have broken a link somewhere, but right now a lot of links are broken.

Next steps

There is a branch here with an example we can try and emulate: https://github.com/ConvivioTeam/paas-tech-docs/tree/GTD-14-prototype-multipage-navigation/source/troubleshooting/troubleshooting1

This is a list of general resources. It should be discoverable from the
resources section but needs cleaning up a bit.
@MatMoore
Copy link
Contributor Author

cc @deborahchua

@MatMoore
Copy link
Contributor Author

Here's the issue for middleman hanging and a workaround: alphagov/tech-docs-template#149

@MatMoore
Copy link
Contributor Author

Closing in favour of #108

@MatMoore MatMoore closed this Oct 12, 2018
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.

1 participant