Skip to content

feat(brand): bunny logo across favicon, README banner, and social preview#81

Merged
naorsabag merged 3 commits into
masterfrom
favicon-rabbit
May 7, 2026
Merged

feat(brand): bunny logo across favicon, README banner, and social preview#81
naorsabag merged 3 commits into
masterfrom
favicon-rabbit

Conversation

@naorsabag
Copy link
Copy Markdown
Owner

@naorsabag naorsabag commented May 7, 2026

Summary

Brand-pass adding the pixel-art bunny + carrots logo (OPENHOP wordmark) across every visual surface that didn't yet have it:

  • Favicon (already in 73b7fb9) — the bunny actor sprite is now the browser tab icon for the local web UI and the GitHub Pages deploy.
  • README banner — logo PNG centered right under the `

    OpenHop

    ` title, above the existing flow screenshot. Matches the layout pattern of comparable dev-tool repos (Storybook, Tailwind, Remotion).
  • Social preview source asset — same image at `.github/social-preview.png` (1280×640, 0.77 MB, under GitHub's 1 MB cap), ready for Settings → Social preview upload. The actual upload is a manual web-UI step (GitHub's API doesn't expose a setter).
  • README casing repair — two CI/stargazers badge URLs had reverted to PascalCase (`naorsabag/OpenHop`); fixed back to lowercase to match the repo's canonical name post-rename.

Linked context

  • Audit's W18 (custom social preview) — this PR provides the source asset; the manual Settings upload step still completes the loop and flips `usesCustomOpenGraphImage` to `true`.
  • Audit's W4 (README spec gaps per `03-readme-and-landing.md`) — partially closed: the "logo + tagline" header pattern from `03:35` is now in place.

Testing notes

  • `docs/logo.png` and `.github/social-preview.png` are byte-identical (verified via `cp`); same image at the same dimensions, just referenced from two locations.
  • README renders correctly on GitHub: logo banner, tagline, anchor nav, existing flow screenshot, badges. No layout regression.
  • All `naorsabag/openhop` URLs in the README are lowercase; `grep -nE "naorsabag/(OpenHop|openhop)" README.md` shows only lowercase hits.
  • Image dimensions confirmed: `1280×640`, PNG, 807 KB. Center-cropped from a 1536×1024 source to GitHub's recommended 2:1 aspect ratio.

Checklist

  • Build passes (`npm run build`)
  • Docs updated if user-facing behavior changed
  • `CHANGELOG.md` entry added under `Unreleased` if user-facing — branding only, will batch with v0.1.0 release notes
  • Manual follow-up: upload `.github/social-preview.png` via Settings → Social preview to flip `usesCustomOpenGraphImage: true`. Verify with `gh api graphql -f query='query { repository(owner:"naorsabag", name:"openhop") { openGraphImageUrl usesCustomOpenGraphImage } }'`.

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Documentation

    • Updated branding references and repository links across documentation
    • Added a dedicated security reporting link for vulnerability disclosures
  • Chores

    • Optimized favicon loading by switching from inline data URI to external file reference

naorsabag and others added 2 commits May 6, 2026 16:47
Drops the inline-emoji 🔀 placeholder and the unused purple-bolt
favicon.svg. The new favicon reuses the canvas's bunny actor sprite
(public/sprites/user_node.svg) — same character + palette as the
animation, so the tab icon previews what the app actually renders.

The source viewBox is 640x624; padded to 640x640 for a square favicon.

Index.html now references /favicon.svg by URL instead of inlining the
SVG, so Vite's base-prefix rewriting works on the Pages deploy.
- Insert pixel-art OpenHop logo (bunny + carrots) as a centered banner
  right after the README title, above the existing flow screenshot.
- Add the same image at .github/social-preview.png as the source asset
  for GitHub's Settings → Social preview upload.
- Repair PascalCase repo URLs that had crept back into the README
  (CI badge, stargazers link).

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 7, 2026

Important

Review skipped

Review was skipped due to path filters

⛔ Files ignored due to path filters (2)
  • .github/social-preview.png is excluded by !**/*.png
  • docs/logo.png is excluded by !**/*.png

CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including **/dist/** will override the default block on the dist directory, by removing the pattern from both the lists.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: d8327941-e3d6-41f6-a7a9-bec99d5eecb2

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Walkthrough

Two independent changes: README.md repository links and branding rebranded from OpenHop to openhop with new security advisories URL, and favicon configuration in packages/web/index.html switched from inline data-URI to external SVG file.

Changes

Branding and Repository Link Updates

Layer / File(s) Summary
Documentation & Badges
README.md
CI/npm/license badges and "Run from source" contributor command updated to use lowercase naorsabag/openhop GitHub repository URL.
Security Link
README.md
Added "Security reports" link pointing to naorsabag/openhop/security/advisories.

Favicon Configuration

Layer / File(s) Summary
Favicon Reference
packages/web/index.html
Favicon <link> tag changed from inline base64 data-URI SVG to external /favicon.svg file reference with image/svg+xml type.

Estimated Code Review Effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly Related PRs

  • naorsabag/openhop#80: Modifies the same packages/web/index.html file with an identical favicon.svg external reference change.
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 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 changes: applying a bunny logo across favicon, README banner, and social preview assets.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch favicon-rabbit

Warning

Review ran into problems

🔥 Problems

Git: Failed to clone repository. Please run the @coderabbitai full review command to re-trigger a full review. If the issue persists, set path_filters to include or exclude specific files.

Tip

💬 Introducing Slack Agent: The best way for teams to turn conversations into code.

Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.

  • Generate code and open pull requests
  • Plan features and break down work
  • Investigate incidents and troubleshoot customer tickets together
  • Automate recurring tasks and respond to alerts with triggers
  • Summarize progress and report instantly

Built for teams:

  • Shared memory across your entire org—no repeating context
  • Per-thread sandboxes to safely plan and execute work
  • Governance built-in—scoped access, auditability, and budget controls

One agent for your entire SDLC. Right inside Slack.

👉 Get started


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

The previous logo was generated by flattening the RGBA source onto an
implicit gray (~120,120,120) canvas via .convert('RGB'), making the
background opaque gray on dark themes. Re-crop and resize while keeping
the alpha channel so the corners stay alpha=0.

File size also drops 808 KB → 257 KB (RGBA-with-sparse-pixels compresses
better than fully-opaque RGB).

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@naorsabag naorsabag merged commit 49aa324 into master May 7, 2026
7 checks passed
@naorsabag naorsabag deleted the favicon-rabbit branch May 8, 2026 06:30
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.

1 participant