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

Alphabetize reordered data source menu #1539

Open
wants to merge 8 commits into
base: dev
Choose a base branch
from

Conversation

nmdefries
Copy link
Contributor

@nmdefries nmdefries commented Sep 3, 2024

Summary:

Context

When using the nav_order setting in markdown front matter, pages sharing the same nav_order are alphabetized by filename, NOT by title. For existing pages, change filenames to alphabetize properly but specify permalink to keep URLs the same.

(For new pages, filenames should be set to match page titles so that alphabetization is correct. Adding permalinks is unnecessary.

We could also force alphabetization using nav_order, but this approach minimizes future work. That is, we won't have to keep inserting new nav_order values between existing ones. All new inactive data sources will be assigned nav_order: 2; active ones will have nav_order: 1.)

Also mention "NHSN" in hhs documentation.

Prerequisites:

  • Unless it is a documentation hotfix it should be merged against the dev branch
  • Branch is up-to-date with the branch to be merged with, i.e. dev
  • Build is successful
  • Code is cleaned up and formatted

Copy link

sonarcloud bot commented Sep 3, 2024

@nmdefries nmdefries requested a review from tinatownes September 3, 2024 21:54
Base automatically changed from ndefries/menu-cleanup to dev September 9, 2024 19:18
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