Skip to content

Replace img tags with Next.js Image component - #28

Merged
hunterbecton merged 2 commits into
mainfrom
refactor/use-next-image
May 1, 2026
Merged

hunterbecton merged 2 commits into
mainfrom
refactor/use-next-image

Conversation

@hunterbecton

@hunterbecton hunterbecton commented May 1, 2026 •

Copy link
Copy Markdown
Member

Summary

  • Replace all <img> elements with next/image <Image> in the demo and docs apps for automatic image optimization (srcset, lazy loading, WebP/AVIF format negotiation)
  • Add object-cover CSS to evolution chain sprites so artwork properly fills the circular container
  • Add images.remotePatterns config to docs next.config.mjs for PokeAPI images
  • Add next/image test mock and wire it into all affected test files

Test plan

  • All 75 demo tests pass
  • Visually verify Pokemon cards, hero, detail view, and evolution chain render correctly
  • Verify skeleton loading states still work (bone("block") spread provides transparent pixel src)
  • Verify docs demo pokemon card renders correctly

Use next/image for automatic optimization (srcset, lazy loading,
format negotiation) in the demo and docs apps. Add object-cover
to evolution chain sprites so artwork fills the circular container
instead of clipping to the top-left corner.
@vercel

vercel Bot commented May 1, 2026 •

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
bones-demo Ready Ready Preview, Comment May 1, 2026 5:40pm
bones-docs Ready Ready Preview, Comment May 1, 2026 5:40pm

Request Review

@hunterbecton
hunterbecton merged commit 73e8a31 into main May 1, 2026
11 checks passed

This branch was successfully deployed

2 active deployments
Preview – bones-demo — 52bca1cd Deployed May 1, 2026 by vercel[bot]
Preview – bones-docs — 52bca1cd Deployed May 1, 2026 by vercel[bot]
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.

1 participant