Skip to content

chore: added new logo in all variants (supports both colors)#1876

Merged
corywatilo merged 6 commits intomainfrom
logos-v2
Apr 23, 2026
Merged

chore: added new logo in all variants (supports both colors)#1876
corywatilo merged 6 commits intomainfrom
logos-v2

Conversation

@corywatilo
Copy link
Copy Markdown
Contributor

Added all the logo variants we might want into a single component:

<Logo wordmark={false} />
<Logo code={false} />
<Logo />
<Logo code={false} stacked />
<Logo stacked />

Examples

image image

Usage

image image

@greptile-apps
Copy link
Copy Markdown

greptile-apps Bot commented Apr 23, 2026

Comments Outside Diff (1)

  1. apps/code/src/renderer/assets/logos-landscape.tsx, line 215-217 (link)

    P2 Unused clip0_417_409 clipPath

    clip0_417_409 is defined in <defs> but never referenced anywhere in the SVG. It's a superfluous part — can be removed.

    Prompt To Fix With AI
    This is a comment left during a code review.
    Path: apps/code/src/renderer/assets/logos-landscape.tsx
    Line: 215-217
    
    Comment:
    **Unused `clip0_417_409` clipPath**
    
    `clip0_417_409` is defined in `<defs>` but never referenced anywhere in the SVG. It's a superfluous part — can be removed.
    
    How can I resolve this? If you propose a fix, please make it concise.
Prompt To Fix All With AI
This is a comment left during a code review.
Path: apps/code/src/renderer/assets/logos-landscape.tsx
Line: 19

Comment:
**`clip-path` must be `clipPath` in JSX**

React does not pass hyphenated attribute names through to SVG elements — it warns "Invalid DOM property `clip-path`. Did you mean `clipPath`?" and silently drops the attribute. The result is that the color-gradient logomark group loses its clipping boundary and paths rendered outside the intended area become visible.

```suggestion
          clipPath="url(#clip1_417_409)"
```

How can I resolve this? If you propose a fix, please make it concise.

---

This is a comment left during a code review.
Path: apps/code/src/renderer/assets/logos-landscape.tsx
Line: 215-217

Comment:
**Unused `clip0_417_409` clipPath**

`clip0_417_409` is defined in `<defs>` but never referenced anywhere in the SVG. It's a superfluous part — can be removed.

How can I resolve this? If you propose a fix, please make it concise.

---

This is a comment left during a code review.
Path: apps/code/src/renderer/features/onboarding/components/WelcomeScreen.tsx
Line: 127-133

Comment:
**Hidden subtitle is dead code**

The "Your product workbench." `<Text>` is permanently hidden via `className="hidden"`. If the subtitle is no longer needed, removing the element entirely would be cleaner than rendering dead JSX.

How can I resolve this? If you propose a fix, please make it concise.

Reviews (1): Last reviewed commit: "stacked variations" | Re-trigger Greptile

@corywatilo corywatilo changed the title chore: added landscape logo (supports both colors) chore: added new logo in all variants (supports both colors) Apr 23, 2026
@corywatilo corywatilo merged commit 7692b9f into main Apr 23, 2026
15 checks passed
@corywatilo corywatilo deleted the logos-v2 branch April 23, 2026 22:03
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