Repository navigation
fix: use shared Avatar in comparison table - #264
sulimanshams wants to merge 2 commits into
Conversation
|
@sulimanshams is attempting to deploy a commit to the osama's projects Team on Vercel. A member of the Team first needs to authorize it. |
|
Thank you for the pull request! ✅ A maintainer will review this soon. Please be patient while we take a look. 🙌 |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
O2sa
left a comment
There was a problem hiding this comment.
One small cleanup before merging: please remove the unused Image import from next/image and the commented-out duplicate import. Also, please confirm that the shared fallback works for missing or whitespace-only avatar URLs in the comparison table.
Thanks for the review! I removed the unused |
Summary:
Replace the direct next/image usage in the comparison table with the shared Avatar component.
Preserve the 40px avatar size and existing accessibility text.
Reuse the shared component's fallback behavior when an avatar URL is missing.
Testing:
pnpm lint
pnpm test -- --run
Related issue: #240