Skip to content

dhi: update customization#24500

Merged
craig-osterhout merged 1 commit intodocker:mainfrom
craig-osterhout:engdocs-3218
Mar 24, 2026
Merged

dhi: update customization#24500
craig-osterhout merged 1 commit intodocker:mainfrom
craig-osterhout:engdocs-3218

Conversation

@craig-osterhout
Copy link
Copy Markdown
Contributor

Description

The UI wording is always Packages.

Related issues or tickets

ENGDOCS-3218

Reviews

  • Editorial review

Signed-off-by: Craig Osterhout <craig.osterhout@docker.com>
@netlify
Copy link
Copy Markdown

netlify bot commented Mar 24, 2026

Deploy Preview for docsdocker ready!

Name Link
🔨 Latest commit 2a373de
🔍 Latest deploy log https://app.netlify.com/projects/docsdocker/deploys/69c2a8e97eb98e0008135bb0
😎 Deploy Preview https://deploy-preview-24500--docsdocker.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@craig-osterhout craig-osterhout requested a review from a team March 24, 2026 15:08
@craig-osterhout craig-osterhout added the status/review Pull requests that are ready for review label Mar 24, 2026
Copy link
Copy Markdown
Contributor

@docker-agent docker-agent bot left a comment

Choose a reason for hiding this comment

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

Assessment: 🟢 APPROVE

This PR simplifies the UI wording in the Docker Hardened Images customization guide. The change removes the distribution-specific distinction (Alpine vs Debian) and consistently uses "Packages" for the drop-down label, which matches the actual UI.

Changes reviewed:

  • ✅ Simplified instruction text for better clarity
  • ✅ Proper UI element formatting with bold (**Packages**)
  • ✅ Consistent with style guide (uses "select" not "click")
  • ✅ No markdown formatting issues

No issues found. The documentation accurately reflects the UI and follows the style guide.

@craig-osterhout craig-osterhout merged commit 266f40c into docker:main Mar 24, 2026
19 checks passed
craig-osterhout added a commit that referenced this pull request Mar 24, 2026
<!--Delete sections as needed -->

## Description

Follow up to #24500. I incorrectly interpreted and tested only the
distros and not the distro versions.

The UI wording is dynamic based on distro *and* version. I removed the
UI wording instead of updating as it makes the simple step more complex
than necessary by having to explain an insignificant UI quirk. Added the
explanation to the paragraph below the main step text.

Debian UI wording is **Packages**
Alpine 3.22 UI wording is **Packages**
Alpine 3.23 UI wording is **Hardened Packages**

## Related issues or tickets

ENGDOCS-3218

## Reviews

<!-- Notes for reviewers here -->
<!-- List applicable reviews (optionally @tag reviewers) -->

- [ ] Editorial review

Signed-off-by: Craig Osterhout <craig.osterhout@docker.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/dhi status/review Pull requests that are ready for review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants