Skip to content

replace check circle icon with animated svg#1340

Merged
joshunrau merged 1 commit intoDouglasNeuroInformatics:mainfrom
david-roper:animate-checkmark
Apr 22, 2026
Merged

replace check circle icon with animated svg#1340
joshunrau merged 1 commit intoDouglasNeuroInformatics:mainfrom
david-roper:animate-checkmark

Conversation

@david-roper
Copy link
Copy Markdown
Collaborator

@david-roper david-roper commented Apr 22, 2026

closes issue #1228

Assisted with claude code
model: Claude Sonnet 4.6

Summary by CodeRabbit

Release Notes

  • Improvements
    • Updated the session success indicator with animated visual effects. The icon now displays with a progressive animation sequence where the circle element animates first, followed by the checkmark element with staggered timing for a more polished visual presentation.

@david-roper david-roper requested a review from joshunrau as a code owner April 22, 2026 18:52
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 22, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 338dce7f-27e5-48aa-adbd-b7520feb11cd

📥 Commits

Reviewing files that changed from the base of the PR and between 7dbc438 and 18b7e4d.

📒 Files selected for processing (1)
  • apps/web/src/routes/_app/session/start-session.tsx

Walkthrough

The start-session.tsx route component's success icon was changed from an external lucide-react CheckCircle component to an inline SVG with framer-motion animations. The SVG renders a circle and checkmark with sequential draw animations (circle: 0.5s; checkmark: 0.5s delay + 0.4s duration).

Changes

Cohort / File(s) Summary
Success Icon Animation
apps/web/src/routes/_app/session/start-session.tsx
Replaced external icon dependency with inline SVG success indicator using motion components for animated circle and checkmark paths.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@joshunrau joshunrau merged commit a1dac3b into DouglasNeuroInformatics:main Apr 22, 2026
1 of 2 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.

2 participants