Skip to content

ui: render the waving mascot at 44px (follow-up to #2654) - #2655

Merged
jjramirezn merged 1 commit into
mainfrom
hotfix-user-interview-cta
Aug 10, 2026
Merged

ui: render the waving mascot at 44px (follow-up to #2654)#2655
jjramirezn merged 1 commit into
mainfrom
hotfix-user-interview-cta

Conversation

@abalinda

@abalinda abalinda commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

Summary

Follow-up to #2654 (merged at the "team"-copy commit while this was in flight): render the user-interview card's waving mascot at 44px instead of the clamped 32px.

The shared icon container is size-8 and the Tailwind preflight img { max-width: 100% } clamps any larger logo back to 32px — so the bump rides the existing iconContainerClassName prop (tailwind-merge lets size-11 override size-8). No shared-component changes. The asset already animates (animated webp; ANIM chunk verified through /_next/image, native build is unoptimized).

Task

TASK-21189 — user interviews PR (follow-up)

Risks

None — 2 files, campaign card + its dev preview only. Same back-merge-to-dev debt as #2654 (one back-merge after this lands covers both).

Screenshots

Current captures on the shared assets branch already show the 44px mascot:

Card Catalogue context
card context

Assets branch pr-assets-2654 — delete after this merges.

QA

Local gate green on the commit (prettier · tsc · 2886 jest · CI 8/8 green on the branch push). /dev/home-ctas shows the 44px variant.

Summary by CodeRabbit

  • Style
    • Improved the user-interview carousel CTA by enlarging its logo and updating the icon container for better visual presentation.
    • Applied the updated logo sizing consistently across the preview and carousel views.

The shared icon container is size-8, and the Tailwind preflight img
max-width clamps any larger logo back to 32px - so the size bump rides
the existing iconContainerClassName prop (tailwind-merge lets size-11
override size-8). The asset already animates: it is the animated webp,
which Next serves untouched (ANIM chunk verified through /_next/image).
@vercel

vercel Bot commented Aug 10, 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 Aug 10, 2026 2:59pm

Request Review

@coderabbitai

coderabbitai Bot commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

Warning

Review limit reached

You’ve reached a temporary PR review limit under our Fair Usage Limits Policy.

Your recent review volume is higher than typical usage, so adaptive limits are currently applied.

Next review available in: 28 minutes

Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available.
You're only billed for reviews past your plan's rate limits ($0.25/file).

How can I continue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews.

How do review limits work?

CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability.

For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window.

Please refer docs for additional details.

Review details
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 95820412-0ab7-4b61-8ed5-21d0a561fcbf

📥 Commits

Reviewing files that changed from the base of the PR and between 11601d0 and 2c83ad7.

📒 Files selected for processing (2)
  • src/app/(mobile-ui)/dev/home-ctas/page.tsx
  • src/hooks/useHomeCarouselCTAs.tsx
📝 Walkthrough

Walkthrough

The user-interview CTA logo increases from 30px to 44px in the development preview and home carousel. Both locations use a size-11 icon container.

Changes

User-interview CTA

Layer / File(s) Summary
Enlarge user-interview CTA logo
src/app/(mobile-ui)/dev/home-ctas/page.tsx, src/hooks/useHomeCarouselCTAs.tsx
The preview and carousel CTA use a 44px logo and a size-11 icon container.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

Suggested labels: enhancement

Suggested reviewers: hugo0

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the main change: rendering the waving mascot at 44px.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch hotfix-user-interview-cta

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

@abalinda

Copy link
Copy Markdown
Contributor Author

@coderabbitai review

@coderabbitai

coderabbitai Bot commented Aug 10, 2026

Copy link
Copy Markdown
Contributor
✅ Action performed

Review finished.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@github-actions

Copy link
Copy Markdown
Contributor

Code-analysis diff

Painscore total: 7141.68 → 7141.97 (+0.29)
Findings: 0 net (+7 new, -7 resolved)

🆕 New findings (7)

  • critical complexity — src/hooks/useHomeCarouselCTAs.tsx — CC 69, MI 57.11, SLOC 316
  • high hotspot — src/hooks/useHomeCarouselCTAs.tsx — 38 commits, +451/-270 lines since 6 months ago
  • high method-complexity — src/hooks/useHomeCarouselCTAs.tsx:147 — CC 30 SLOC 143
  • medium high-mdd — src/hooks/useHomeCarouselCTAs.tsx:82 — useHomeCarouselCTAs: MDD 170.8 (uses across many lines from declarations)
  • medium high-mdd — src/hooks/useHomeCarouselCTAs.tsx:147 — : MDD 122.8 (uses across many lines from declarations)
  • medium complexity — src/app/(mobile-ui)/dev/home-ctas/page.tsx — CC 7, MI 56.2, SLOC 93
  • low missing-return-type — src/app/(mobile-ui)/dev/home-ctas/page.tsx:173 — HomeCTAsPreviewPage: exported fn missing return type annotation

✅ Resolved (7)

  • src/hooks/useHomeCarouselCTAs.tsx — CC 69, MI 57.14, SLOC 315
  • src/hooks/useHomeCarouselCTAs.tsx — 37 commits, +447/-269 lines since 6 months ago
  • src/hooks/useHomeCarouselCTAs.tsx:147 — CC 30 SLOC 142
  • src/hooks/useHomeCarouselCTAs.tsx:82 — useHomeCarouselCTAs: MDD 168.3 (uses across many lines from declarations)
  • src/hooks/useHomeCarouselCTAs.tsx:147 — : MDD 120.6 (uses across many lines from declarations)
  • src/app/(mobile-ui)/dev/home-ctas/page.tsx — CC 7, MI 56.3, SLOC 92
  • src/app/(mobile-ui)/dev/home-ctas/page.tsx:172 — HomeCTAsPreviewPage: exported fn missing return type annotation

@github-actions

Copy link
Copy Markdown
Contributor

🧪 UI test report — ✅ all green

Suites

  • unit: 2890 ran, 0 failed, 0 skipped, 46.1s

📊 Coverage (unit)

metric %
statements 66.0%
branches 50.9%
functions 56.0%
lines 66.7%
⏱ 10 slowest test cases
time test
3.0s src/components/Card/share-asset/__tests__/shareAssetLayout.test.ts › never places two stickers in heavy overlap (broad seed sweep)
1.1s src/utils/__tests__/demo-api.test.ts › isDemoMode() is false when not running under Capacitor
0.3s src/utils/__tests__/sentry.utils.test.ts › defaults to the client budget under a browser global
0.3s src/app/(mobile-ui)/withdraw/__tests__/withdraw-states.test.tsx › Bank withdrawal keeps the $1 minimum for sub-$1 amounts
0.3s src/utils/__tests__/sentry.utils.test.ts › still lets a per-call timeoutMs win over the default
0.3s src/utils/__tests__/auth-token.test.ts › is none — never guarded — when only the guarded marker is present
0.3s src/utils/__tests__/demo-balance.test.ts › keeps a spent-down balance across a cold start within the TTL
0.3s 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 validateInviteCode body
0.2s src/utils/__tests__/demo-balance.test.ts › auto-refills a stored balance that has no timestamp (legacy install)
📍 Inline annotations are in the **Unit test report** check above. Coverage artifact: `coverage-unit`. Generated by `.github/workflows/tests.yml`.

@abalinda
abalinda marked this pull request as ready for review August 10, 2026 15:02
@abalinda
abalinda requested review from Hugo0 and jjramirezn August 10, 2026 15:08
@abalinda

Copy link
Copy Markdown
Contributor Author

@coderabbitai full review

@coderabbitai

coderabbitai Bot commented Aug 10, 2026

Copy link
Copy Markdown
Contributor
⚠️ Action not completed

Review rate limited.


Your included review limit is currently reached under our Fair Usage Limits Policy. Your recent PR review activity is in the 95th percentile or higher among CodeRabbit users, so adaptive limits apply. This review may still proceed through usage-based billing if eligible. Your next included review will be available in 28 minutes.

@abalinda
abalinda requested a lite review from Copilot August 10, 2026 15:39

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@jjramirezn
jjramirezn merged commit aa6fee2 into main Aug 10, 2026
28 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.

3 participants