Skip to content

Fix invisible GitHub logo in the integrations list - #1547

Merged
RhysSullivan merged 2 commits into
UsefulSoftwareCo:mainfrom
midego1:fix/github-logo-visibility
Aug 28, 2026
Merged

Fix invisible GitHub logo in the integrations list#1547
RhysSullivan merged 2 commits into
UsefulSoftwareCo:mainfrom
midego1:fix/github-logo-visibility

Conversation

@midego1

@midego1 midego1 commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

The GitHub logo is invisible in the integrations list and in the connect dialog on the light theme.

Both GitHub presets (openapi "GitHub REST" and graphql "GitHub GraphQL") point at svgl's github_dark.svg. That is the dark-background variant — its paths are filled #ffff, i.e. white. On the light UI that renders a white mark on a white surface, so the row just looks blank. openai_dark.svg has the identical problem, so it is fixed in the same commit.

Swapping to github_light.svg would only move the bug to the dark theme: preset icons render as a plain <img src={preset.icon}> with no theme-aware variant selection, so a single URL has to be legible on both.

Instead this points both at https://integrations.sh/logo/<domain> — which integration-favicon.tsx describes as "executor's single logo source", and which the neighbouring Figma, Stripe and Vercel presets already use. Those assets carry their own background, so they stay readable on either theme.

Verified the replacements resolve: integrations.sh/logo/github.com → 200 image/png (the standard dark-chip Octocat mark), integrations.sh/logo/openai.com → 200 image/svg+xml.

midego1 and others added 2 commits August 6, 2026 08:51
The GitHub presets pointed at svgl's `github_dark.svg`, which is the
dark-BACKGROUND variant: its paths are filled `#ffff`. Rendered on the light
UI that is a white mark on a white surface, so the GitHub entry in the
integrations list and in the connect dialog looked blank. `openai_dark.svg`
has the same problem.

Point both at integrations.sh instead, which `integration-favicon.tsx` calls
executor's single logo source and which the neighbouring Figma, Stripe and
Vercel presets already use. Those logos carry their own background, so they
stay legible on either theme -- preset icons render as a plain `<img>` with
no theme-aware variant selection, so one URL has to work on both.
@RhysSullivan
RhysSullivan merged commit 349b049 into UsefulSoftwareCo:main Aug 28, 2026
40 checks passed
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