Update partner logos for CodeRabbit and Cloudflare#1145
Update partner logos for CodeRabbit and Cloudflare#1145piecyk merged 2 commits intoTanStack:mainfrom
Conversation
|
📝 WalkthroughWalkthroughThe README's partner logos section updates HTML picture elements for CodeRabbit and Cloudflare with new srcset asset filenames for dark/light theme variants, modified src image references, removal of height attributes from source tags, and removal of padding from the Cloudflare cell. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
View your CI Pipeline Execution ↗ for commit 06e9866
☁️ Nx Cloud last updated this comment at |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@README.md`:
- Around line 71-74: The fallback <img> currently uses the white Cloudflare
asset which will look bad on light backgrounds; update the <img> inside the
picture block (the <img src> fallback) to use the black/light variant URL
(https://tanstack.com/assets/cloudflare-black-6Ojsn8yh.svg) so clients that
ignore <picture><source> render a light-mode-safe image (you can also add a
srcset attribute mirroring the light variant if desired).
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
| <source media="(prefers-color-scheme: dark)" srcset="https://tanstack.com/assets/cloudflare-white-Co-Tyjbl.svg" /> | ||
| <source media="(prefers-color-scheme: light)" srcset="https://tanstack.com/assets/cloudflare-black-6Ojsn8yh.svg" /> | ||
| <img src="https://tanstack.com/assets/cloudflare-white-Co-Tyjbl.svg" height="60" alt="Cloudflare" /> | ||
| </picture> |
There was a problem hiding this comment.
Use a light-mode-safe fallback image for Cloudflare.
On Line 73, the fallback <img src> is the white asset. In clients that don’t apply <picture><source>, this can render poorly on light backgrounds. Prefer the black/light variant as fallback.
Suggested fix
- <img src="https://tanstack.com/assets/cloudflare-white-Co-Tyjbl.svg" height="60" alt="Cloudflare" />
+ <img src="https://tanstack.com/assets/cloudflare-black-6Ojsn8yh.svg" height="60" alt="Cloudflare" />📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| <source media="(prefers-color-scheme: dark)" srcset="https://tanstack.com/assets/cloudflare-white-Co-Tyjbl.svg" /> | |
| <source media="(prefers-color-scheme: light)" srcset="https://tanstack.com/assets/cloudflare-black-6Ojsn8yh.svg" /> | |
| <img src="https://tanstack.com/assets/cloudflare-white-Co-Tyjbl.svg" height="60" alt="Cloudflare" /> | |
| </picture> | |
| <source media="(prefers-color-scheme: dark)" srcset="https://tanstack.com/assets/cloudflare-white-Co-Tyjbl.svg" /> | |
| <source media="(prefers-color-scheme: light)" srcset="https://tanstack.com/assets/cloudflare-black-6Ojsn8yh.svg" /> | |
| <img src="https://tanstack.com/assets/cloudflare-black-6Ojsn8yh.svg" height="60" alt="Cloudflare" /> | |
| </picture> |
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@README.md` around lines 71 - 74, The fallback <img> currently uses the white
Cloudflare asset which will look bad on light backgrounds; update the <img>
inside the picture block (the <img src> fallback) to use the black/light variant
URL (https://tanstack.com/assets/cloudflare-black-6Ojsn8yh.svg) so clients that
ignore <picture><source> render a light-mode-safe image (you can also add a
srcset attribute mirroring the light variant if desired).
🎯 Changes
✅ Checklist
pnpm run test:pr.🚀 Release Impact
Summary by CodeRabbit