Repository navigation
Bring the app in line with the landing page's look - #41
Merged
Merged
Conversation
Headlines in DM Sans 400 with tight tracking, one navy (#14243b), navy pill buttons (light pills in dark mode), the website's white highlight behind Home's amount, uppercase 12px labels and 24px surface corners. Sage stays for settled states. Most of it sits in styles/harmony.css. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01HcVKDmy3nYR8taPWrTXANu
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01HcVKDmy3nYR8taPWrTXANu
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01HcVKDmy3nYR8taPWrTXANu
EmotiveImpact
marked this pull request as ready for review
October 12, 2026 16:13
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. |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Owner
Author
|
The Vercel preview for
I can't read this deployment's log because Vercel refused access (403 on the Generated by Claude Code |
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01HcVKDmy3nYR8taPWrTXANu
The space above the date and the space between the sentence and the next-step card are now the same (28px). Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01HcVKDmy3nYR8taPWrTXANu
This branch had an error being 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.
Before: the web app and the landing page looked like two products. The app used heavy Inter headlines, its own navy (#1a1e2b), square-cornered buttons and 15px cards.
After: the web app takes on the landing page's look. Headlines use light DM Sans with tight tracking. There is one navy, the landing page's #14243b. Buttons are round and keep the app's green. Home's amount sits in the landing page's white highlight. The date and group headings are small uppercase labels, and cards have 24px corners. The phone app (apps/native) also gets round green buttons and a round filter on Expenses.
Edem chose the green and the round shape on 12 October. Layouts, wording, money rules and the database are unchanged, and the landing page itself is not touched.
Before and after screenshots: https://claude.ai/artifact/E49jAN7K6ErEcyTbCSLUSy
How:
apps/web/src/styles/harmony.css, in the overrides layer.shared.css:--navy,--navy-2,--navy-3,--radius-surface, and a new--font-display.design-system.css.theme-colorand the manifest use the new navy.stylesheets.test.tsanddesign-tokens.test.tswere updated to match.Buttoninsrc/ui/kit.tsxusesradius.pill.Checks:
tscand the Vite build pass.tscand its 8 tests pass.🤖 Generated with Claude Code
https://claude.ai/code/session_01HcVKDmy3nYR8taPWrTXANu
Generated by Claude Code