Skip to content

[plugins] Pad selected plugin logos#326

Merged
ssetty-oai merged 3 commits into
mainfrom
dev/ssetty/plugin-icon-padding
Jun 9, 2026
Merged

[plugins] Pad selected plugin logos#326
ssetty-oai merged 3 commits into
mainfrom
dev/ssetty/plugin-icon-padding

Conversation

@ssetty-oai

@ssetty-oai ssetty-oai commented Jun 9, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Add 192x192 padded icon/logo assets for the plugin icons called out by openai/openai#996735 that are owned in this repo: Base44, fal, Figma, OpenAI Developers, Picsart, Shutterstock, Shopify, Supabase, and Vercel.
  • Point the changed plugins' logo fields at the padded assets, and point remaining tight composerIcon fields at the same padded assets where present.
  • Keep Vercel on a fully white 192x192 canvas so its white app-icon background is extended consistently.
  • Leave Browser, Databricks Genie, Sites, and Snowflake untouched because matching plugin directories are not present in this repository.

Validation

  • jq empty on the changed plugin manifests
  • git diff --check
  • Targeted script verifying generated PNG outputs and manifest icon paths
  • Rendered Shopify and Supabase SVG outputs locally with rsvg-convert for visual inspection

Related client-side workaround: https://github.com/openai/openai/pull/996735

@ssetty-oai ssetty-oai marked this pull request as ready for review June 9, 2026 21:04
@ssetty-oai ssetty-oai requested a review from a team June 9, 2026 21:04
@ssetty-oai ssetty-oai changed the title [codex] Pad selected plugin logos [plugins] Pad selected plugin logos Jun 9, 2026
@ssetty-oai ssetty-oai marked this pull request as draft June 9, 2026 22:52
@ssetty-oai ssetty-oai marked this pull request as ready for review June 9, 2026 23:09
@ssetty-oai ssetty-oai merged commit c6ea566 into main Jun 9, 2026
@ssetty-oai ssetty-oai deleted the dev/ssetty/plugin-icon-padding branch June 9, 2026 23:55
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.

2 participants