Skip to content

fix(card): bring launch CTA in line with the activation CTAs#2299

Merged
Hugo0 merged 2 commits into
mainfrom
hotfix/card-launch-cta-design-system
Jun 29, 2026
Merged

fix(card): bring launch CTA in line with the activation CTAs#2299
Hugo0 merged 2 commits into
mainfrom
hotfix/card-launch-cta-design-system

Conversation

@Hugo0

@Hugo0 Hugo0 commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

What

Bring the in-app card launch CTA (CardLaunchCTABanner) in line with the other activation CTAs, per Konrad's launch-day design review.

3 changes (cosmetic only):

  • Pink bg-primary-1bg-white (pink reads as "clickable" everywhere else in Peanut; this is a card)
  • Removed the shadow-[4px_4px_0_#000] drop shadow (black-on-black, and it misaligned the bounding box)
  • Button variant="dark"variant="purple" (the standard primary CTA; the black CTA exists nowhere else)

Copy ("shhh" / "Tap to find out if you're in" / "Try the door →"), the /shhhhh routing, and all gating are unchanged.

Why hotfix to main

The launch (#2298 + api #1079) is already merged and deployed; this is a cosmetic follow-up to match the design system without waiting for the next dev→main cycle.

Follow-up

Needs back-merging to dev after this merges (standard release-flow) so the next release doesn't reintroduce the old styling.

Risk

Single file, 5 lines, no logic/routing/gating change. Prettier confirmed class order unchanged.

Launch-day design review (Konrad): the banner was off-system — pink (pink=clickable in Peanut), a black drop-shadow that misaligned the bounding box, and a black CTA used nowhere else. Now a white card, no drop shadow, standard purple primary CTA — matching the other activation CTAs. Copy + /shhhhh routing + gating unchanged; cosmetic only.
@vercel

vercel Bot commented Jun 29, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
peanut-wallet Ready Ready Preview, Comment Jun 29, 2026 10:52am

Request Review

@coderabbitai

coderabbitai Bot commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 1294753e-b48b-4cb5-ae9d-c8c5c5e14162

📥 Commits

Reviewing files that changed from the base of the PR and between 323fa61 and 8d78395.

📒 Files selected for processing (1)
  • src/components/Home/CardLaunchCTA/CardLaunchCTABanner.tsx
🚧 Files skipped from review as they are similar to previous changes (1)
  • src/components/Home/CardLaunchCTA/CardLaunchCTABanner.tsx

Walkthrough

CardLaunchCTABanner is updated to use a white card with a black border and no shadow, with the CTA button changed to purple and the headline text changed to shhhh.

Changes

CardLaunchCTABanner styling

Layer / File(s) Summary
Banner background, border, and button variant
src/components/Home/CardLaunchCTA/CardLaunchCTABanner.tsx
Comment text reflects the updated visual spec and dismiss behavior; the wrapper switches to a white background with a black border and no shadow; the headline changes to shhhh and the primary button switches from dark to purple.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • peanutprotocol/peanut-ui#2295: Changes the same CardLaunchCTABanner component and its CTA presentation, including button styling and banner layout.
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly summarizes the main cosmetic update to the launch CTA and is concise and specific.
Description check ✅ Passed The description accurately matches the changeset and explains the cosmetic CTA styling update.
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch

Comment @coderabbitai help to get the list of available commands.

@github-actions

github-actions Bot commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

Code-analysis diff

Painscore total: 5846.18 → 5846.47 (+0.29)
Findings: 0 net (+0 new, -0 resolved)

@github-actions

github-actions Bot commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

🧪 UI test report — ✅ all green

Suites

  • unit: 1598 ran, 0 failed, 0 skipped, 23.3s

📊 Coverage (unit)

metric %
statements 54.4%
branches 37.0%
functions 42.0%
lines 54.3%
⏱ 10 slowest test cases
time test
3.9s src/components/Card/share-asset/__tests__/shareAssetLayout.test.ts › never places two stickers in heavy overlap (broad seed sweep)
0.5s src/components/Card/share-asset/__tests__/shareAssetLayout.test.ts › every sticker stays within canvas at any count
0.3s src/app/actions/__tests__/api-headers-extended.test.ts › should not include apiKey in updateUserById body
0.2s src/app/actions/__tests__/api-headers.test.ts › should include Content-Type in updateUserById
0.1s src/components/Global/GeneralRecipientInput/__tests__/GeneralRecipientInput.test.tsx › should handle minimum length (6 digits) US account
0.1s src/components/Request/__tests__/request-states.test.tsx › renders request form with nav header, action card, QR code, amount input, and create button
0.1s src/components/Global/GeneralRecipientInput/__tests__/GeneralRecipientInput.test.tsx › should handle valid 9-digit US account
0.1s src/components/Global/GeneralRecipientInput/__tests__/GeneralRecipientInput.test.tsx › should handle valid US account with spaces 2
0.1s src/components/Global/GeneralRecipientInput/__tests__/GeneralRecipientInput.test.tsx › should handle valid Italian IBAN
0.1s src/components/Global/GeneralRecipientInput/__tests__/GeneralRecipientInput.test.tsx › should handle valid US account with spaces
📍 Inline annotations are in the **Unit test report** check above. Coverage artifact: `coverage-unit`. Generated by `.github/workflows/tests.yml`.

Border was border-2 (2px) vs the activation CTAs' 1px (Global Card = border border-black); drop to 1px. Title shhh → shhhh (one more h).
@Hugo0 Hugo0 merged commit 80ffc2d into main Jun 29, 2026
20 of 23 checks 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.

1 participant