Skip to content

fix(.icons/poolside.svg): use official Poolside logo - #1084

Merged
DevelopmentCats merged 2 commits into
mainfrom
fix/poolside-official-icon
Aug 27, 2026
Merged

fix(.icons/poolside.svg): use official Poolside logo#1084
DevelopmentCats merged 2 commits into
mainfrom
fix/poolside-official-icon

Conversation

@ausbru87

Copy link
Copy Markdown
Contributor

Summary

The coder-labs/pool module (added in #1033) shipped a placeholder icon at .icons/poolside.svg that renders as a lime letterform on a dark square rather than Poolside's actual logo. This replaces it with Poolside's official logo (the spiral mark from poolside.ai), so it displays correctly next to the module on registry.coder.com.

Details

  • Sourced from Poolside's official brand asset (poolside.ai favicon SVG), transparent background, no baked-in tile.
  • Theme-adaptive fill so the mark reads on both registry themes:
    • Light: #4137FF (Poolside primary indigo)
    • Dark: #B9B6FF (Poolside light lavender), via @media (prefers-color-scheme: dark).
  • No module code changes; the pool module already references .icons/poolside.svg via frontmatter (icon:) and var.icon (/icon/poolside.svg).

Before / after

  • Before: lime letter mark on #151515 rounded square (not the Poolside logo).
  • After: Poolside's official spiral logo.

Opened by Coder Agents on behalf of @ausbru87.

ausbru87 and others added 2 commits August 27, 2026 15:57
The Pool CLI module (added in #1033) shipped a placeholder mark that
renders as a lime letter rather than Poolside's logo. Replace it with
Poolside's official logo from poolside.ai, with a theme-adaptive fill
(#4137FF light, #B9B6FF dark) so it reads on both registry themes.
@DevelopmentCats
DevelopmentCats merged commit 61da9d6 into main Aug 27, 2026
4 checks passed
@DevelopmentCats
DevelopmentCats deleted the fix/poolside-official-icon branch August 27, 2026 18:47
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