mirror of
https://github.com/coder/coder.git
synced 2026-06-02 20:48:20 +00:00
docs: address review comments for DOCS_STYLE_GUIDE.md (#21178)
## Summary This PR addresses David's review comments from PR #21153 to improve the Documentation Style Guide. ## Changes - **Research section**: Updated to focus on reading "recent documentation" instead of "10+ similar pages" - more practical guidance - **Premium Feature Callout**: Clarified that the manifest.json badge addition should happen in `docs/manifest.json` - **Screenshot Guidelines**: Added context that this is for when screenshots "don't exist yet", making it clearer this is a temporary measure - **Tabs documentation**: Expanded explanation to clarify when tabs are appropriate (parallel content paths) - **Coder registry**: Added mention of referencing Coder registry URLs for cross-linking to external Coder resources All changes maintain the existing documentation structure while improving clarity and specificity based on review feedback. Refs #21153 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com> --------- Co-authored-by: Claude Sonnet 4.5 <noreply@anthropic.com>
This commit is contained in:
@@ -6,7 +6,7 @@ This guide documents documentation patterns observed in the Coder repository, ba
|
||||
|
||||
Before documenting a feature:
|
||||
|
||||
1. **Research similar documentation** - Read 10+ similar pages in `docs/` to understand writing style, structure, and conventions for your content type (admin guides, tutorials, reference docs, etc.)
|
||||
1. **Research similar documentation** - Read recent documentation pages in `docs/` to understand writing style, structure, and conventions for your content type (admin guides, tutorials, reference docs, etc.)
|
||||
2. **Read the code implementation** - Check backend endpoints, frontend components, database queries
|
||||
3. **Verify permissions model** - Look up RBAC actions in `coderd/rbac/` (e.g., `view_insights` for Template Insights)
|
||||
4. **Check UI thresholds and defaults** - Review frontend code for color thresholds, time intervals, display logic
|
||||
@@ -42,10 +42,7 @@ Template Insights provides detailed analytics and usage metrics for your Coder t
|
||||
|
||||
### Premium Feature Callout
|
||||
|
||||
For Premium-only features:
|
||||
|
||||
1. Add `(Premium)` suffix to the H1 heading - the documentation system automatically links these to premium pricing information
|
||||
2. Add premium badge in `manifest.json` with `"state": ["premium"]`
|
||||
For Premium-only features, add `(Premium)` suffix to the H1 heading. The documentation system automatically links these to premium pricing information. You should also add a premium badge in the `docs/manifest.json` file with `"state": ["premium"]`.
|
||||
|
||||
```markdown
|
||||
# Template Insights (Premium)
|
||||
@@ -94,7 +91,7 @@ When you have multiple screenshots showing different aspects of a feature:
|
||||
|
||||
### Screenshot Guidelines
|
||||
|
||||
**When screenshots are not available**: Use image placeholders with descriptive alt text and ask the user to provide screenshots:
|
||||
**When screenshots are not yet available**: If you're documenting a feature before screenshots exist, you can use image placeholders with descriptive alt text and ask the user to provide screenshots:
|
||||
|
||||
```markdown
|
||||

|
||||
@@ -103,6 +100,7 @@ When you have multiple screenshots showing different aspects of a feature:
|
||||
Then ask: "Could you provide a screenshot of the Template Insights page? I've added a placeholder at [location]."
|
||||
|
||||
**When documenting with screenshots**:
|
||||
|
||||
- Illustrate features being discussed in preceding text
|
||||
- Show actual UI/data, not abstract concepts
|
||||
- Reference specific values shown when explaining features
|
||||
@@ -133,7 +131,7 @@ Then ask: "Could you provide a screenshot of the Template Insights page? I've ad
|
||||
- `> [!NOTE]` for additional information
|
||||
- `> [!WARNING]` for important warnings
|
||||
- `> [!TIP]` for helpful tips
|
||||
- **Tabs**: Use tabs to present related but parallel documentation paths (e.g., different installation methods, platform-specific instructions)
|
||||
- **Tabs**: Use tabs for presenting related but parallel content, such as different installation methods or platform-specific instructions. Tabs work well when readers need to choose one path that applies to their specific situation.
|
||||
|
||||
## Writing Style
|
||||
|
||||
@@ -162,19 +160,19 @@ Then ask: "Could you provide a screenshot of the Template Insights page? I've ad
|
||||
|
||||
### Command Examples
|
||||
|
||||
```markdown
|
||||
````markdown
|
||||
```sh
|
||||
coder server --disable-template-insights
|
||||
```
|
||||
```
|
||||
````
|
||||
|
||||
### Environment Variables
|
||||
|
||||
```markdown
|
||||
````markdown
|
||||
```sh
|
||||
CODER_DISABLE_TEMPLATE_INSIGHTS=true
|
||||
```
|
||||
```
|
||||
````
|
||||
|
||||
### Code Comments
|
||||
|
||||
@@ -191,6 +189,8 @@ Use relative paths from current file location:
|
||||
- `[Template Permissions](./template-permissions.md)`
|
||||
- `[API documentation](../../reference/api/insights.md)`
|
||||
|
||||
For cross-linking to Coder registry templates or other external Coder resources, reference the appropriate registry URLs.
|
||||
|
||||
### Cross-References
|
||||
|
||||
- Link to related documentation at the end
|
||||
|
||||
Reference in New Issue
Block a user