Repository navigation
Conversation
The logo used a darker green (#4b8e33) than the app's primary color (#66c02a), and two of its paths referenced gradients that were never defined. The logo, favicons, and email header now use the app's green, with #6bc72d on dark backgrounds. Emails use the same green. Buttons on it use dark text for contrast, and links use #3c7d18 so they stay readable on white. Slack attachments use the new green too. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
A white E on the brighter green was hard to read at tab size (2.3:1). The favicon and small logo now use a dark E (8.9:1), matching the dark text used on green email buttons, and drop the old gloss and sparkle. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Keeps the dark E for contrast and restores the white sparkle from the original icon. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The icon was a traced bitmap of the old glossy art. It is now a vector built from the wordmark's E, with a clean four-point sparkle in the top-left corner and a subtle green gradient. - Add favicon.svg, keeping a 48px PNG fallback, and an Apple touch icon - The legacy favicon.ico now has 16, 32, and 48px sizes, with the 16px sparkle placed by hand so it stays crisp - The mobile navbar logo uses the SVG Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The sparkle sat apart from the letter, so it read as a stray mark. It now glints off the E's top-left corner with a soft glow, and the 16px version places it on the corner by hand. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The E now fills most of the tile, and a small sparkle glints off its very top-left corner instead of covering the letter. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The previous redraw used the wordmark's narrower E. The icon now uses the original icon's E and tile shapes from exceptionless/MediaKit icon.svg, with the sparkle redrawn as a vector in its original spot at the E's top-left. Colors are the new green with a dark E for contrast. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The thin-ray glint was hard to see at small sizes. The icon now has a solid four-point AI sparkle cut into the E's top-left corner, where the original glint sat, with a small gap so it reads as part of the mark. The 16px version places the sparkle by hand. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The swoosh from the x now ends in a four-point AI sparkle instead of a round dot, matching the sparkle on the app icon. The email header logo is regenerated at 2x so it stays sharp on high-DPI screens. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The gap around the sparkle used a gradient scaled to its own shape, so it showed as a slightly darker outline against the tile. It now uses the tile's gradient coordinates. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
2 of 27 tasks
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Refreshes the brand ahead of the new marketing site and docs screenshots. The logo used a darker green (
#4b8e33) than the app's primary color (hsl(96 64% 46%), about#66c02a), so the logo now matches the app instead of the other way around. The logo and app icon also pick up a four-point AI sparkle, since Exceptionless is leaning into AI (Exie and the MCP server).Changes
logo.svg,logo-dark.svg): the mark is now#66c02a(light) and#6bc72d(dark), and the dot over the i is a four-point AI sparkle, so the swoosh from the x ends in the sparkle. Two paths referenced gradients that were never defined (url(#prefix__prefix__a)andurl()), so they now use explicit fills.favicon.svg,exceptionless-icon.svg) from the original icon's E and tile shapes in MediaKiticon.svg, with the original glint replaced by a solid four-point AI sparkle cut into the E's top-left corner. The tile uses the new green and the E is dark: a white E on the brighter green is only 2.3:1 and hard to read at tab size; the dark E is 8.9:1.app.htmlnow links the SVG favicon, a 48px PNG fallback, and a new 180pxapple-touch-icon.png.wwwroot/favicon.icohas 16, 32, and 48px sizes, with the 16px sparkle placed by hand so it stays crisp.exceptionless-48.png, which is removed fromsrc/lib/assets.static/img/exceptionless-48.pngstays (email metadata links to it) and is regenerated.exceptionless-logo.png): regenerated from the new logo at 2x (470x100), so it stays sharp at its 235x50 display size on high-DPI screens.#66c02a, and buttons on it use dark text (#1a1a1a) for contrast.#3c7d18so they stay readable on white.PrimaryActionandActionLinkare removed, and footer action links useLink.#66C02A.wwwroot/img/logo.pngandwwwroot/img/exceptionless.pngare unchanged because emails that were already sent link to them.Testing
Mailertests pass (35) with the same diff on this base commit.npm run checkpasses (0 errors); checked the favicon, Apple touch icon, and mobile navbar icon in light and dark.🤖 Generated with Claude Code