Skip to content

Use brand logomark in site header#1

Merged
greynewell merged 1 commit intomainfrom
fix/use-brand-logomark
Feb 12, 2026
Merged

Use brand logomark in site header#1
greynewell merged 1 commit intomainfrom
fix/use-brand-logomark

Conversation

@robotson
Copy link
Contributor

@robotson robotson commented Feb 12, 2026

Summary

  • Replaces generic Lucide layers icon with the actual Supermodel eye/diamond logomark SVG
  • SVG sourced from the dashboard's logomark.svg
  • Uses fill="currentColor" to inherit text color and hover state

Test plan

  • Built site locally with go run generate-index.go
  • Verified logo renders correctly at http://localhost:8888

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Style
    • Refreshed the site header logo with an enhanced, larger design featuring improved visual detail and clarity.

Uses the actual eye/diamond SVG from the dashboard (logomark.svg) instead of a
generic Lucide layers icon. Fill set to currentColor so it inherits text/hover colors.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@coderabbitai
Copy link

coderabbitai bot commented Feb 12, 2026

Walkthrough

The SVG logo in the index template was updated from a 24x24 outlined design with multiple paths to a larger 90x78 single-path version with an xmlns attribute, purely affecting static HTML rendering without changing any runtime logic.

Changes

Cohort / File(s) Summary
Logo SVG Update
generate-index.go
Replaced site brand SVG markup with a larger, more detailed single-path design; added xmlns attribute to the SVG element.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🎨 A logo grew from small to grand,
From outlined paths to one more planned,
The header now wears bigger pride,
With xmlns standing by its side! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: replacing the generic layers icon with the brand logomark in the site header.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/use-brand-logomark

No actionable comments were generated in the recent review. 🎉

Tip

Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord.


Comment @coderabbitai help to get the list of available commands and usage tips.

@greynewell greynewell merged commit 189bb60 into main Feb 12, 2026
1 of 2 checks passed
greynewell added a commit that referenced this pull request Feb 14, 2026
Sync SVG icon across all templates to match the brand logomark
from main (#1), replacing the generic layers icon.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@coderabbitai coderabbitai bot mentioned this pull request Feb 14, 2026
4 tasks
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

Comments