Skip to content

chore: migrate icons from Material Symbols to Heroicons#25051

Open
dvdksn wants to merge 1 commit into
docker:mainfrom
dvdksn:heroicons
Open

chore: migrate icons from Material Symbols to Heroicons#25051
dvdksn wants to merge 1 commit into
docker:mainfrom
dvdksn:heroicons

Conversation

@dvdksn
Copy link
Copy Markdown
Contributor

@dvdksn dvdksn commented May 13, 2026 •
edited
Loading

Summary

  • Replaces `@material-symbols/svg-400` (5.5 MB) with `heroicons` (488 KB) — an 11x reduction in icon dependency size
  • Heroicons' single consistent format (24px solid SVGs, `fill="currentColor"`) lets us delete the separate `utils/svg.html` partial, the `icon-svg-stroke` CSS utility, and the dual-path resolution logic in `icon.html`
  • All icon references across layouts, shortcodes, and content front matter updated to kebab-case Heroicons equivalents; three custom icons (gordon, gordon-happy, system) kept as bespoke SVGs in `assets/icons/`

🤖 Generated with Claude Code

@netlify
Copy link
Copy Markdown

netlify Bot commented May 13, 2026 •
edited
Loading

✅ Deploy Preview for docsdocker ready!

Name Link
🔨 Latest commit 07d9975
🔍 Latest deploy log https://app.netlify.com/projects/docsdocker/deploys/6a045e8f057acc00075d0721
😎 Deploy Preview https://deploy-preview-25051--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.

@github-actions github-actions Bot added area/engine Issue affects Docker engine/daemon area/compose Relates to docker-compose.yml spec or docker-compose binary area/hub Issue affects Docker Hub area/build Relates to Dockerfiles or docker build command area/desktop Issue affects a desktop edition of Docker. E.g docker for mac area/security area/get-started Relates to get started and onboarding docs area/subscription Relates to Docker pricing and subscription area/extensions Relates to Docker Extensions area/admin Relates to Docker Admin area/scout Relates to Docker Scout hugo Updates related to hugo dependencies Vendoring, packaging updates area/billing area/accounts Relates to Docker accounts area/config area/guides area/ai area/dhi area/offload labels May 13, 2026
@dvdksn dvdksn requested a review from a team May 13, 2026 11:04
@dvdksn dvdksn added the status/review Pull requests that are ready for review label May 13, 2026
@dvdksn @claude
chore: migrate icons from Material Symbols to Heroicons
07d9975 
Replaces @material-symbols/svg-400 (5.5 MB) with heroicons (488 KB) — an
11x reduction in icon asset size. Heroicons uses a single consistent
distribution format (24px solid SVGs, fill="currentColor") eliminating
the need for the separate utils/svg.html partial, the icon-svg-stroke CSS
utility, and the dual-path resolution logic in icon.html.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@docker-agent
Copy link
Copy Markdown

docker-agent commented May 13, 2026

⏱️ PR Review Timed Out — The review agent hit the 1800 s time limit before completing. This usually happens on large or complex diffs. You can re-trigger with /review — if it times out again, consider splitting the PR into smaller pieces.

@dvdksn
Copy link
Copy Markdown
Contributor Author

dvdksn commented May 13, 2026

/review

@docker-agent
Copy link
Copy Markdown

docker-agent commented May 13, 2026

⏱️ PR Review Timed Out — The review agent hit the 1800 s time limit before completing. This usually happens on large or complex diffs. You can re-trigger with /review — if it times out again, consider splitting the PR into smaller pieces.

@craig-osterhout
Copy link
Copy Markdown
Contributor

craig-osterhout commented May 13, 2026

Quick glance and the icon replacements lgtm.
The top of the manual pages seems incorrect for Ask Docs AI and Open in Claude.

image

@dvdksn
Copy link
Copy Markdown
Contributor Author

dvdksn commented May 15, 2026

@craig-osterhout oops good catch, bad rebase!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@craig-osterhout craig-osterhout Awaiting requested review from craig-osterhout craig-osterhout is a code owner

@aevesdocker aevesdocker Awaiting requested review from aevesdocker aevesdocker is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

area/accounts Relates to Docker accounts area/admin Relates to Docker Admin area/ai area/billing area/build Relates to Dockerfiles or docker build command area/compose Relates to docker-compose.yml spec or docker-compose binary area/config area/desktop Issue affects a desktop edition of Docker. E.g docker for mac area/dhi area/engine Issue affects Docker engine/daemon area/extensions Relates to Docker Extensions area/get-started Relates to get started and onboarding docs area/guides area/hub Issue affects Docker Hub area/offload area/scout Relates to Docker Scout area/security area/subscription Relates to Docker pricing and subscription dependencies Vendoring, packaging updates hugo Updates related to hugo status/review Pull requests that are ready for review

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@dvdksn @docker-agent @craig-osterhout