refactor: reorganize and enhance the "Why HTTPS matters" section in c… #256
Conversation
…ertificate configuration documentation
There was a problem hiding this comment.
Pull request overview
This PR refactors the structure of the certificate configuration documentation by moving the "Why HTTPS matters" section from a top-level heading to a subsection and removing the "Overview" section header.
Changes:
- Moved "Why HTTPS matters" section from its position before "Overview" to after the introductory content and caution aside
- Changed "Why HTTPS matters" from an H2 heading (##) to an H3 heading (###)
- Removed the "Overview" H2 section header entirely
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| included in publish or deployment artifacts. | ||
| </Aside> | ||
|
|
||
| ### Why HTTPS matters |
There was a problem hiding this comment.
The "Why HTTPS matters" section is now an H3 heading (###) but appears before any H2 heading in the document. This creates an orphaned subsection that doesn't belong to any parent section. According to proper Markdown document hierarchy, H3 headings should only appear under H2 headings. Consider either keeping it as an H2 heading or creating a parent H2 section for it.
| ### Why HTTPS matters | |
| ## Why HTTPS matters |
…ertificate configuration documentation
Today, this article has two "Overview" headings, we there's an implicit one for the top of the article. I slightly reorganized this to have the actual overview be first, then why HTTPS matters as a subsection.