Shorten Learn the Basics sidebar headings#823
Conversation
Trim the within-page H2s on Explaining ESPHome and What is YAML? so the left sidebar entries don't wrap or feel unwieldy. - ESPHome vs ESPHome Device Builder -> ESPHome vs Device Builder - How you talk to your device -> Controlling your device - Why the Starter Kit is built on ESPHome -> Why ESPHome - What this looks like in ESPHome -> ESPHome example
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (3)
WalkthroughThree ESPHome Starter Kit learning documentation pages are cleaned up with simplified heading names, improved YAML guidance, and consistent removal of redundant "Back to First Steps" navigation buttons to streamline the learning experience. ChangesDocumentation Structure and Navigation Updates
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Poem
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Customer feedback flagged 'surfaces' as jargon and asked for stronger coverage of YAML spacing rules. - Replace 'This surfaces your device's sensors, switches, and lights as entities' with a plainer 'Each of your device's sensors, switches, and lights then appears in Home Assistant as an entity' - New 'Spacing matters' H2 with the three core rules (no tabs, consistent indent, space after colon) and a correct vs broken example - Trim the 'Indentation groups things together' H4 lead-in now that the spaces vs tabs detail lives in the new section - Remove the 'Common gotchas' H4, now fully covered by Spacing matters
Customer feedback suggested adding advice to copy the existing YAML into a notepad before hand-editing, as a safety net if a bad indent breaks the build. Lands at the end of Spacing matters where the failure mode just got explained.
These pages can be reached from anywhere in the sidebar or via inbound links, so a Back to First Steps button at the bottom assumes a path that may not exist. The remaining buttons are the external resource and cross-link CTAs.
What does this implement/fix?
Trims the within-page H2 headings on Explaining ESPHome and What is YAML? so the left sidebar entries are shorter and don't wrap.
ESPHome vs ESPHome Device Builder->ESPHome vs Device BuilderHow you talk to your device->Controlling your deviceWhy the Starter Kit is built on ESPHome->Why ESPHomeWhat this looks like in ESPHome->ESPHome exampleBody content is unchanged. Holding this PR open for more changes before merge.
Types of changes
mkdocs.yml)Checklist:
devbranch (notmain)mkdocs servemkdocs.ymlmkdocs.ymlSummary by CodeRabbit