fix(react): replace world icons with human badge - #331
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 63d9c68815
ℹ️ 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".
| import humanBadgeIcon from "./HumanBadgeIcon.svg"; | ||
|
|
||
| export function HumanBadgeIcon(): ReactElement { | ||
| return <img alt="" src={humanBadgeIcon} width={32} height={32} />; |
There was a problem hiding this comment.
Render the badge without a data-URL dependency
When an integrator enforces a CSP such as img-src 'self' (or inherits that restriction from default-src) without allowing data:, the tsup loader converts this import into a data:image/svg+xml,... URL that the browser blocks, so the badge disappears from every connection header and app-opening button. The previous inline SVG was unaffected by img-src; keeping the badge inline would preserve compatibility with restrictive relying-party CSPs.
Useful? React with 👍 / 👎.
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
This PR replaces the World logo throughout the React widget with the verified human badge from Figma.