Skip to content

Align auth shell logo with Chloei#12

Merged
chloeilabs merged 1 commit into
mainfrom
codex/align-auth-shell-with-chloei
Apr 19, 2026
Merged

Align auth shell logo with Chloei#12
chloeilabs merged 1 commit into
mainfrom
codex/align-auth-shell-with-chloei

Conversation

@chloeilabs
Copy link
Copy Markdown
Owner

@chloeilabs chloeilabs commented Apr 19, 2026

Summary

  • restore the shared logo animation stylesheet in finance
  • restore full-size logo SVG sizing so the auth shell matches chloei
  • add a focused regression test for the logo SVG output

Verification

  • pnpm exec vitest run src/components/graphics/logo/tests/logo-svg.test.tsx src/components/auth/tests/auth-shell.test.tsx 'src/app/(auth)/tests/sign-in-page.test.tsx' 'src/app/(auth)/tests/sign-up-page.test.tsx'
  • curl -s http://localhost:3000/sign-in | rg -n "size-full fill-current|logo-md|Finance|Sign In"

Summary by CodeRabbit

  • New Features

    • Added slideshow animation effect to logos with varying speeds based on size.
  • Tests

    • Added comprehensive test suite for logo SVG components.
  • Style

    • Updated logo sizing for improved layout consistency.

@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 19, 2026

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

Project Deployment Actions Updated (UTC)
finance Ready Ready Preview, Comment Apr 19, 2026 3:53am

Request Review

@chatgpt-codex-connector
Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 19, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 6077748d-9ff5-4639-ba7d-17eb2fb072f6

📥 Commits

Reviewing files that changed from the base of the PR and between 3dbe829 and f600f26.

📒 Files selected for processing (7)
  • src/components/graphics/logo/__tests__/logo-svg.test.tsx
  • src/components/graphics/logo/logo-animation.css
  • src/components/graphics/logo/logo-burst-svg.tsx
  • src/components/graphics/logo/logo-burst.tsx
  • src/components/graphics/logo/logo-hover-svg.tsx
  • src/components/graphics/logo/logo-hover.tsx
  • src/components/graphics/logo/logo-svg.tsx

📝 Walkthrough

Walkthrough

Added CSS keyframes animation (logo-slideshow) for logo components triggered by data-animate attribute. Updated three SVG components to include size-full Tailwind class. Added stylesheet imports to parent components and created test suite verifying animation markers and styling in rendered output.

Changes

Cohort / File(s) Summary
Logo SVG Components
src/components/graphics/logo/logo-svg.tsx, src/components/graphics/logo/logo-hover-svg.tsx, src/components/graphics/logo/logo-burst-svg.tsx
Added size-full Tailwind class to each component's root <svg> element's className, maintaining existing fill-current and user-provided overrides via cn(...).
Logo Animation Integration
src/components/graphics/logo/logo-burst.tsx, src/components/graphics/logo/logo-hover.tsx
Added imports for new ./logo-animation.css stylesheet to enable animation behavior; no changes to component logic or exported signatures.
Animation Styles
src/components/graphics/logo/logo-animation.css
Introduced @keyframes logo-slideshow with horizontal translateX translation from 0% to -100%. Added animation rules for .logo-sm, .logo-xs, .logo-md (1.5s) and .logo-lg (2s) triggered by [data-animate="true"] attribute using steps(15) infinite animation.
Test Suite
src/components/graphics/logo/__tests__/logo-svg.test.tsx
Added Vitest test suite verifying rendered SVG components include expected class="size-full fill-current" and confirm data-animate attribute presence on hover and burst variants.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 Hopping through styles with CSS so grand,
Logos now animate across the land,
Size-full classes dance in perfect time,
Tests confirm the animated sublime!
Slideshow magic, step by step they go—
Watch our bunny logos steal the show! 🎬✨

✨ 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 codex/align-auth-shell-with-chloei

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

@chloeilabs chloeilabs merged commit bce6b33 into main Apr 19, 2026
5 of 6 checks passed
@chloeilabs chloeilabs deleted the codex/align-auth-shell-with-chloei branch April 19, 2026 03:54
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