Skip to content

fix(open-in): update Termy logo#2893

Merged
arnestrickmann merged 1 commit into
mainfrom
jan/update-termy-logo
Jul 16, 2026
Merged

fix(open-in): update Termy logo#2893
arnestrickmann merged 1 commit into
mainfrom
jan/update-termy-logo

Conversation

@janburzinski

@janburzinski janburzinski commented Jul 16, 2026

Copy link
Copy Markdown
Collaborator

Description

updated termy.sh logo

Screenshot/Recording (if applicable)

Updated Termy logo

Checklist
  • I kept this PR small and focused
  • I ran a self-review before opening this PR
  • I ran the relevant local checks or explained why not
  • I updated docs when behavior or setup changed (not applicable)
  • I added or updated tests when behavior changed, or explained why not
  • I only added comments where the logic is not obvious
  • I used Conventional Commits for commit messages and, when possible, the PR title

@janburzinski
janburzinski marked this pull request as ready for review July 16, 2026 12:03
@greptile-apps

greptile-apps Bot commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR updates the Termy logo used by the desktop app.

  • Replaces the Termy PNG with a self-contained SVG.
  • Updates the shared icon path to reference the SVG.

Confidence Score: 5/5

This looks safe to merge.

  • No blocking issues found in the changed code.
  • The new path matches the committed asset.
  • Existing SVG icons use the same Vite and Electron rendering path.

Important Files Changed

Filename Overview
apps/emdash-desktop/src/assets/images/termy.svg Adds a valid, self-contained SVG logo supported by the existing asset pipeline.
apps/emdash-desktop/src/assets/images/termy.png Removes the superseded Termy PNG asset.
apps/emdash-desktop/src/shared/openInApps.ts Changes the Termy icon mapping to the new SVG filename.

Reviews (1): Last reviewed commit: "fix(open-in): update Termy logo" | Re-trigger Greptile

@arnestrickmann
arnestrickmann merged commit 3968ba9 into main Jul 16, 2026
1 check passed
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