-
-
Notifications
You must be signed in to change notification settings - Fork 210
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.indexes feature to mkdocs-material #1101
Conversation
Signed-off-by: Adam Warner <[email protected]>
…add redirect. Remove images that are not referenced anywhere else Signed-off-by: Adam Warner <[email protected]>
…edirects. Replace references to renamed pages Signed-off-by: Adam Warner <[email protected]>
Signed-off-by: Adam Warner <[email protected]>
Signed-off-by: Adam Warner <[email protected]>
Signed-off-by: Adam Warner <[email protected]>
Signed-off-by: Adam Warner <[email protected]>
Signed-off-by: Adam Warner <[email protected]>
✅ Deploy Preview for pihole-docs ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it is a bit inconsistently used: some item suse the index feature, others not (e.g. About Pihole, Getting Started). I think it should behave the same way everywhere.
That's why I suggested the change above. |
Signed-off-by: Adam Warner <[email protected]>
The difficulty here is inventing content for the index pages of those sections. I think the only ones where this appears inconsistent now are:
|
Signed-off-by: Adam Warner <[email protected]>
Signed-off-by: Adam Warner <[email protected]>
What does this PR aim to accomplish?:
Discussed internally, effectively saves a click by making linking the index to the main section header rather than having a subheader for overview/index.
I've purposefully left the Docker section as I will be revamping that entire section in another PR
By submitting this pull request, I confirm the following:
git rebase
)