Skip to content

feat(auth): drop "Create Account" heading; CADAM wordmark on sign-in - #207

Merged
aaron-at-adam merged 1 commit into
masterfrom
cadam-signin-signup-cleanup
Jun 17, 2026
Merged

feat(auth): drop "Create Account" heading; CADAM wordmark on sign-in#207
aaron-at-adam merged 1 commit into
masterfrom
cadam-signin-signup-cleanup

Conversation

@aaron-at-adam

@aaron-at-adam aaron-at-adam commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

Summary

Small follow-up to the CADAM rebrand (#202, #206) on the auth pages.

Changes

  • Sign-up (/signup) — removes the "Create Account" heading so the CADAM wordmark stands alone as the page header.
  • Sign-in — swaps the remaining adam-logo-full.svg for the CADAM wordmark (cadam-logo.svg). This swap was made in feat(home): add "the new Adam" announcement banner #202 but dropped in its squash-merge, so the live sign-in page was still showing the Adam logo. Now matches sign-up.

Files

  • src/views/SignUpView.tsx — drop the heading
  • src/views/SignInView.tsx — CADAM wordmark

Checks

  • tsc -b --noEmit passes; pre-commit eslint + prettier clean.

🤖 Generated with Claude Code


Summary by cubic

Aligns auth pages with the CADAM rebrand. Removes the "Create Account" heading on /signup and swaps the sign-in logo to cadam-logo.svg with "CADAM Logo" alt text, replacing the last Adam logo.

Written for commit 46aae22. Summary will update on new commits.

Review in cubic

- Sign-up (/signup): remove the "Create Account" heading so the CADAM
  wordmark stands alone as the header.
- Sign-in: swap the remaining adam-logo-full.svg for the CADAM wordmark
  (this swap was lost in the #202 squash merge), matching sign-up.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@vercel

vercel Bot commented Jun 17, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
cadam Ready Ready Preview, Comment Jun 17, 2026 7:41am

Request Review

@supabase

supabase Bot commented Jun 17, 2026

Copy link
Copy Markdown

This pull request has been ignored for the connected project sgprnbvihmydyrzvkcir because there are no changes detected in supabase directory. You can change this behaviour in Project Integrations Settings ↗︎.


Preview Branches by Supabase.
Learn more about Supabase Branching ↗︎.

@aaron-at-adam
aaron-at-adam merged commit 03498eb into master Jun 17, 2026
5 checks passed
@greptile-apps

greptile-apps Bot commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

Rebrand follow-up that swaps the remaining adam-logo-full.svg reference in SignInView.tsx for cadam-logo.svg and removes the "Create Account" h1 from SignUpView.tsx. The sign-in logo swap is clean, but the heading removal was only applied to the main signup view — SignUpEmailView.tsx still renders the same heading above the same wordmark.

  • SignInView.tsx: Logo src and alt text updated correctly; cadam-logo.svg is present in /public/.
  • SignUpView.tsx: h1 removed, but SignUpEmailView.tsx is a parallel signup path that still has <h1>Create Account</h1> (line 126), leaving the two flows visually inconsistent. The now-single-child container also retains an orphaned gap-2.

Confidence Score: 4/5

Safe to merge, but the heading removal is incomplete — the email signup flow will look different from the main signup flow until SignUpEmailView.tsx is also updated.

The sign-in logo swap is straightforward and correct. The sign-up heading removal works on the main flow but was not applied to SignUpEmailView.tsx, which is a parallel user-facing signup path with the same wordmark + heading layout. Users who land on the email signup flow will still see 'Create Account' as a heading while users on the main flow won't.

src/views/SignUpView.tsx — the intent is right but the companion file SignUpEmailView.tsx was missed.

Important Files Changed

Filename Overview
src/views/SignInView.tsx Swaps adam-logo-full.svg for cadam-logo.svg and updates alt text; cadam-logo.svg exists in /public/. Clean change.
src/views/SignUpView.tsx Removes "Create Account" h1, but the parallel email signup view (SignUpEmailView.tsx) still has the same heading — visual inconsistency between the two signup flows. Also leaves an orphaned gap-2 on the now single-child container.

Reviews (1): Last reviewed commit: "feat(auth): drop "Create Account" headin..." | Re-trigger Greptile

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.

1 participant