Skip to content

nav: add groupings for manuals #21471

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 1 commit into from
Closed

nav: add groupings for manuals #21471

wants to merge 1 commit into from

Conversation

@craig-osterhout
Copy link
Contributor

@craig-osterhout craig-osterhout commented Nov 19, 2024 •
edited
Loading

Description

Adds nav groups, specifically for manuals.
Allows for another layer of structure without hiding the children.

https://deploy-preview-21471--docsdocker.netlify.app/manuals/

Add 'group: "something"' to the top-level topic (_index.md for sections) in manuals to create a group.
Groups are dynamic and not defined.
It should use weight to sort for both group and within a group.

Currently have the groups as:

  • Products
  • Open source tools
  • Platform
  • Move? (Temp group. Content that doesn't fit the above groups and should be handled)
    • I'm planning to move trusted content back into Hub very soon.
    • Maybe move extensions into Desktop and SDK into references?
    • Maybe move Cloud integrations into some deprecated/retired topic with other old products or into Compose?

Not sure about release lifecycle under products, because it applies to everything. Maybe a reference topic?

Aligned the manuals' landing page with the groups.

Related issues or tickets

Reviews

  • Editorial review

@github-actions github-actions bot added area/engine Issue affects Docker engine/daemon area/compose Relates to docker-compose.yml spec or docker-compose binary area/hub Issue affects Docker Hub area/build Relates to Dockerfiles or docker build command area/desktop Issue affects a desktop edition of Docker. E.g docker for mac area/security area/subscription Relates to Docker pricing and subscription area/extensions Relates to Docker Extensions area/admin Relates to Docker Admin area/scout Relates to Docker Scout hugo Updates related to hugo area/billing area/accounts Relates to Docker accounts area/build-cloud area/copilot labels Nov 19, 2024
@netlify
Copy link

netlify bot commented Nov 19, 2024 •
edited
Loading

✅ Deploy Preview for docsdocker ready!

Name Link
🔨 Latest commit 286017c
🔍 Latest deploy log https://app.netlify.com/sites/docsdocker/deploys/673d127a66649800088215c2
😎 Deploy Preview https://deploy-preview-21471--docsdocker.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@craig-osterhout craig-osterhout changed the title nav: add groupings for manual nav: add groupings for manuals Nov 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@aevesdocker aevesdocker Awaiting requested review from aevesdocker aevesdocker will be requested when the pull request is marked ready for review aevesdocker is a code owner

@dvdksn dvdksn Awaiting requested review from dvdksn dvdksn will be requested when the pull request is marked ready for review dvdksn is a code owner

Assignees

No one assigned

Labels

area/accounts Relates to Docker accounts area/admin Relates to Docker Admin area/billing area/build Relates to Dockerfiles or docker build command area/build-cloud area/compose Relates to docker-compose.yml spec or docker-compose binary area/copilot area/desktop Issue affects a desktop edition of Docker. E.g docker for mac area/engine Issue affects Docker engine/daemon area/extensions Relates to Docker Extensions area/hub Issue affects Docker Hub area/scout Relates to Docker Scout area/security area/subscription Relates to Docker pricing and subscription hugo Updates related to hugo

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@craig-osterhout