Skip to content

fix(react): replace world icons with human badge - #331

Merged
Takaros999 merged 2 commits into
mainfrom
takis/react-human-badge
Sep 9, 2026
Merged

fix(react): replace world icons with human badge#331
Takaros999 merged 2 commits into
mainfrom
takis/react-human-badge

Conversation

@Takaros999

Copy link
Copy Markdown
Contributor

This PR replaces the World logo throughout the React widget with the verified human badge from Figma.

  • Use the badge in connection headers and app-opening buttons, preserving existing sizes.
  • Bundle the exported SVG and remove both obsolete World icon components.
  • Validation: React type check, full build, and all 35 tests pass.

@vercel

vercel Bot commented Sep 4, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated
idkit-js-example Ready Ready Preview Sep 9, 2026 10:11pm UTC

Request Review

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 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} />;

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge 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 👍 / 👎.

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review Completed 2026-09-04T21:49:50.016098Z 63d9c68 PR opened
ℹ️ 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" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@Takaros999
Takaros999 merged commit f20fea5 into main Sep 9, 2026
18 checks passed
@Takaros999
Takaros999 deleted the takis/react-human-badge branch September 9, 2026 22:36
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.

3 participants