Add Clawi to Foundry, rename diVine to Divine#10
Add Clawi to Foundry, rename diVine to Divine#10NotThatKindOfDrLiz wants to merge 1 commit intomainfrom
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 06b71101f8
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| <article class="project-card"> | ||
| <div class="project-icon"> | ||
| <img src="/logos/Divine-512.png" alt="diVine logo" /> | ||
| <img src="/logos/Divine-512.png" alt="Divine logo" /> |
There was a problem hiding this comment.
Compress the new Divine icon before shipping it
The replacement public/logos/Divine-512.png behind this <img> is now ~996 KB at 1024×1024, but the card still renders it inside the 48×48 .project-icon box defined at index.html:755-767. Because the homepage eagerly loads every Foundry logo, this swap alone adds roughly 760 KB to each visit (raising the page's logo payload from about 1.28 MB to 2.05 MB in this commit), which is a noticeable performance regression on slower/mobile connections. Please re-export or compress this asset closer to its rendered size.
Useful? React with 👍 / 👎.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
06b7110 to
fd7721f
Compare
Summary
Test plan
🤖 Generated with Claude Code