feat: add project logo - #201
Conversation
Adds a logo built from demand's own visual language: the `❯` prompt chevron in the Charm theme's indigo-to-fuchsia range, followed by a green block input cursor on a dark terminal tile. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Central YAML (base), Organization UI (inherited) Review profile: CHILL Plan: Pro Plus Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (1)
📝 WalkthroughWalkthroughREADME.md now shows a centered ChangesREADME presentation
Estimated code review effort: 1 (Trivial) | ~2 minutes Poem
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Greptile SummaryAdds a project logo based on demand’s terminal-prompt visual language and displays it above the README heading.
Confidence Score: 5/5The PR appears safe to merge with no actionable issues identified. The README references an existing added asset, and the SVG is self-contained, structurally valid, and does not alter application behavior. Important Files Changed
Reviews (1): Last reviewed commit: "feat: add project logo" | Re-trigger Greptile |
demand had no logo. This adds one, built from the library's own visual language rather than something generic: the
❯prompt chevron (demand'scursor_str) in the Charm theme's indigo → fuchsia range, followed by a green block input cursor, on a dark terminal tile.All three colors are lifted straight from
Theme::charm()insrc/theme.rs— indigo#7571f9, fuchsia#f780e2, green#02bf87.assets/logo.svg— 512×512,rx=112README.md— centered above the# demandheading at 128pxIt reads cleanly down to 32px, which is the size jdx.dev renders project logos at.
The companion change adding it to jdx.dev is jdx/blog#82.
Note: I had no SVG rasterizer available in this environment, so the file hasn't been checked through a renderer other than the one used to preview the paths — worth a glance at the rendered README before merging.
🤖 Generated with Claude Code
Note
Low Risk
Documentation and static asset only; no application logic, APIs, or dependencies change.
Overview
Adds branding assets for demand: a new
assets/logo.svg(512×512, rounded dark tile) showing a gradient chevron and green block cursor using Charm theme colors fromTheme::charm()(indigo → fuchsia, green#02bf87).The README now shows that logo centered above the
# demandheading at 128×128. No library or runtime code changes.Reviewed by Cursor Bugbot for commit fa2be80. Bugbot is set up for automated code reviews on this repo. Configure here.
Summary by CodeRabbit