Skip to content

feat: unify loading spinners with central LoadingSpinner component (#…#451

Merged
codeZe-us merged 1 commit into
SafeVault:devfrom
Luluameh:feat/442-unified-loading-spinner
Apr 29, 2026
Merged

feat: unify loading spinners with central LoadingSpinner component (#…#451
codeZe-us merged 1 commit into
SafeVault:devfrom
Luluameh:feat/442-unified-loading-spinner

Conversation

@Luluameh
Copy link
Copy Markdown
Contributor

Closes #442

What was changed

  • Created src/components/ui/LoadingSpinner.tsx — central spinner with VestRoll brand color (#5E2A8C), sm/md/lg size variants, fullScreen mode for page-level loading, and ARIA role="status" for accessibility
  • Replaced bare Loader2 in (dashboard)/loading.tsx
  • Replaced hardcoded <div>Loading...</div> Suspense fallback in verify-otp/page.tsx
  • Replaced 3 different inline animate-spin div implementations in login-page, forgot-password-form, and reset-password-form
  • Replaced text-only "Processing..." in Step1BasicInfo, Step2Password, and Step5BusinessDetails

Result

All data-fetching and form submission states across the platform now share the exact same spinning animation and VestRoll branding.

@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 27, 2026

@Luluameh is attempting to deploy a commit to the vestroll's projects Team on Vercel.

A member of the Team first needs to authorize it.

@codeZe-us
Copy link
Copy Markdown
Contributor

@Luluameh this PR points to main branch, it should point to dev not main

@Luluameh Luluameh force-pushed the feat/442-unified-loading-spinner branch from 37e2f53 to d6591fc Compare April 28, 2026 18:54
@Luluameh Luluameh changed the base branch from main to dev April 28, 2026 18:59
@Luluameh
Copy link
Copy Markdown
Contributor Author

@Luluameh this PR points to main branch, it should point to dev not main

Hi
I have updated it. it is pointing to the dev

@codeZe-us codeZe-us self-requested a review April 29, 2026 09:31
@codeZe-us codeZe-us merged commit 6573037 into SafeVault:dev Apr 29, 2026
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Standardize and Unify Loading Spinners

2 participants