Skip to content

feat(icons): hand-drawn goldfish — 8th hand-drawn icon#136

Merged
jacuzzicoding merged 1 commit into
developmentfrom
feat/hand-drawn-goldfish
May 11, 2026
Merged

feat(icons): hand-drawn goldfish — 8th hand-drawn icon#136
jacuzzicoding merged 1 commit into
developmentfrom
feat/hand-drawn-goldfish

Conversation

@jacuzzicoding
Copy link
Copy Markdown
Owner

Summary

  • Adds hand-drawn watercolor + ink goldfish icon (8th in the hand-drawn library)
  • 2048×2048 source committed to icon-sources/fish/goldfish.png
  • Optimized 768×768 deploy asset at public/icons/fish/goldfish.png (88.9 KB, 94.7% reduction via sharp + pngquant)
  • Replaces the scraped wiki placeholder; cross-game routing fans it automatically to any game that maps to fish/goldfish
  • Manifest unchanged — goldfish entry was already present from prior scrape runs

Hand-drawn library sequence: sea-bass, koi, coelacanth, frog, ant, robust-cicada, brown-cicada → goldfish (#8)

Test plan

  • npm run icons:export --force — produced 768×768 8-bit colormap PNG at 88.9 KB
  • npm run build — clean (2.02s, no errors)
  • file public/icons/fish/goldfish.png confirms correct PNG format
  • manifest.json retains goldfish entry

2048px watercolor + ink source committed to icon-sources/fish/goldfish.png.
Replaces the scraped wiki placeholder with the hand-drawn asset at 768px
(88.9 KB, 94.7% reduction via sharp + pngquant). Cross-game routing fans
this automatically to any game that maps to fish/goldfish.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings May 11, 2026 21:46
@vercel
Copy link
Copy Markdown

vercel Bot commented May 11, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
animalcrossingwebapp Ready Ready Preview, Comment May 11, 2026 9:46pm

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@jacuzzicoding jacuzzicoding merged commit 1d8bdcd into development May 11, 2026
3 checks passed
@jacuzzicoding jacuzzicoding deleted the feat/hand-drawn-goldfish branch May 11, 2026 21:47
jacuzzicoding added a commit that referenced this pull request May 12, 2026
…ipeline ref (#137)

Four minimal corrections detected by nightly doc-sync run 2026-05-12:

- CHANGELOG.md: add goldfish (8th hand-drawn icon, PR #136) to [Unreleased]
- docs/roadmap-to-v1.md: hand-drawn progress 7→8 of 255, add goldfish to list,
  date updated to 2026-05-11 (when PR #136 merged)
- public/version-history.html: "seven hand-drawn icons" → "eight", add goldfish
  to the Currently Building checklist item
- CLAUDE.md: icon-sources comment "512 deploy assets" → "768 deploy assets"
  (TARGET_SIZE was bumped in PR #115 / v0.9.4-beta; stale since then)

https://claude.ai/code/session_01MEw8nPdnLDstoNHXVWs1SP

Co-authored-by: Claude <noreply@anthropic.com>
jacuzzicoding pushed a commit that referenced this pull request May 14, 2026
public/roadmap-to-v1.html was generated on 2026-05-12 (PR #138) before the
goldfish hand-drawn icon (PR #136) was recorded in the other doc-sync commit
(94d17ed) that ran 34 seconds later the same day. Result: the HTML roadmap
still shows 7 hand-drawn icons and 203 species remaining; docs/roadmap-to-v1.md
already reflects 8/202.

Corrections:
- Stat tile: 7 / 210  →  8 / 210
- "Where we are" hero number: 7  →  8
- Unique species remaining: 203  →  202  (both the big number and the milestone body)
- Fish list: 4 fish (sea-bass, koi, coelacanth, frog)  →  5 fish (+ goldfish)
- Eyebrow + footer date: 2026-05-12  →  2026-05-14

Note: per-game gameBars data (drawn/scraped counts per game) in the inline
JS block are not updated here — those require running `npm run audit:icons`
to get precise numbers. Left for a future icon-audit pass.

https://claude.ai/code/session_01W5ZSp5SuSQFFU6bQVUZhbU
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants