Skip to content

Conversation

@mafredri
Copy link
Member

@mafredri mafredri commented Dec 9, 2025

Adds screenshots to the dev containers user guide per David's feedback on #21157.

Screenshots added:

  • devcontainer-running.png - Running dev containers as sub-agents with apps, SSH, and ports
  • devcontainer-discovery.png - Discovered dev containers with Start button
  • devcontainer-outdated.png - Outdated status indicator with rebuild option
  • devcontainer-apps-bar.png - Dev container with display apps disabled

Also deletes the outdated devcontainer-agent-ports.png.

Refs #21157

@mafredri mafredri force-pushed the docs/devcontainer-screenshot-placeholders branch 2 times, most recently from 77dd359 to 595a3fb Compare December 9, 2025 15:11
@mafredri
Copy link
Member Author

mafredri commented Dec 9, 2025

@DanielleMaywood @david-fraley how do you feel about adding the images proposed by placeholders? Open to other ideas.

@mafredri mafredri force-pushed the docs/devcontainer-screenshot-placeholders branch 3 times, most recently from 8c1d586 to 1e32637 Compare December 9, 2025 15:20
Copy link
Member Author

Choose a reason for hiding this comment

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

This image is severely outdated.

@mafredri mafredri force-pushed the docs/devcontainer-screenshot-placeholders branch from 1e32637 to 32f6806 Compare December 9, 2025 15:24
@mafredri mafredri force-pushed the docs/devcontainer-screenshot-placeholders branch from 32f6806 to bf9ca78 Compare December 9, 2025 15:28
@DanielleMaywood
Copy link
Contributor

Yeah I think those should be good

Copy link
Collaborator

@david-fraley david-fraley left a comment

Choose a reason for hiding this comment

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

Can you make sure all the images render A-OK before proceeding? You can do that by using this URL -- https://coder.com/docs/@docs%2fdevcontainer-screenshot-placeholders/...

fyi when you navigate by clicking anything in the docs pages, the %2f auto turns into a slash and you'll need to manually change it back

@mafredri mafredri changed the title docs(docs/user-guides/devcontainers): add screenshot placeholders docs: add dev container screenshot placeholders Dec 9, 2025
@mafredri mafredri force-pushed the docs/devcontainer-screenshot-placeholders branch 11 times, most recently from 0795cd8 to 69b1ddd Compare December 9, 2025 17:20
@mafredri mafredri marked this pull request as ready for review December 9, 2025 17:21
@mafredri mafredri requested a review from david-fraley December 9, 2025 17:22
@mafredri mafredri changed the title docs: add dev container screenshot placeholders docs: add dev container screenshots Dec 9, 2025
Add screenshots to the dev containers user guide:

- Running dev containers with sub-agents (index.md, working-with-dev-containers.md)
- Discovered dev containers with Start button (index.md)
- Outdated status with rebuild option (working-with-dev-containers.md)
- Display apps disabled (customizing-dev-containers.md)

Also deletes the outdated devcontainer-agent-ports.png.

Refs #21157
@mafredri mafredri force-pushed the docs/devcontainer-screenshot-placeholders branch from 69b1ddd to 5b668fd Compare December 9, 2025 18:04
@mafredri mafredri enabled auto-merge (squash) December 9, 2025 18:04
@mafredri mafredri merged commit 8f15caa into main Dec 9, 2025
30 checks passed
@mafredri mafredri deleted the docs/devcontainer-screenshot-placeholders branch December 9, 2025 18:09
@github-actions github-actions bot locked and limited conversation to collaborators Dec 9, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants