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

Update dev docs #1243

Merged
merged 2 commits into from
Jun 27, 2023
Merged

Update dev docs #1243

merged 2 commits into from
Jun 27, 2023

Conversation

victorlin
Copy link
Member

Description of proposed changes

Small touch ups. See commit messages.

Related issue(s)

N/A

Testing

N/A

Checklist

  • Add a message in CHANGES.md summarizing the changes in this PR that are end user focused. Keep headers and formatting consistent with the rest of the file. N/A, no functional changes

These were prone to getting out of sync.

For the top-level summary, assume readers are reading the document via
GitHub's markdown preview page and point them to use the interactive
markdown heading navigator.

For the testing section summary, replace it with a one-sentence summary
of the entire section.
The project board is no longer being updated. Replace it with the issue
list, which stays the most up to date.
@victorlin victorlin requested a review from a team June 27, 2023 20:02
- Formats (Markdown and reStructuredText)
- Documentation structure
- Building documentation
Thank you for helping us to improve Augur! Use the GitHub markdown preview sidebar to navigate the sections in this document.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've always missed this from our docs. The original auspice docs had this, and when it works it's amazing (it only half works on GitHub).

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The GitHub preview sidebar works fine for me, but I'm not a huge fan of relying on it.

An embedded table of contents with links to each section (and only up to a certain depth) seems beneficial to me. There's VSCode extensions that automatically insert/maintain one, but that's relying on a tool that not everyone editing this file will use, and I'm not sure those can ignore sections after a certain depth.

I'd be open to manually adding/maintaining one at the top level, something like this. Thoughts?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh, I was referring to page designs which use the right margin to display a floating TOC, and then (bonus) highlight the entry you're currently reading. For example, here's a page built using docusaurus, which was what Auspice initially used for its docs.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh! That does look much nicer. Probably not for this file unless we host it on RTD, but it sounds like some other of our RTD pages could benefit from this feature. Created an issue: nextstrain/docs.nextstrain.org#161

@victorlin victorlin merged commit 45f54da into master Jun 27, 2023
@victorlin victorlin deleted the victorlin/update-dev-docs branch June 27, 2023 21:53
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.

2 participants