Skip to content

feat(ui): replace loading spinners with shimmer skeleton screens#513

Open
fantababa wants to merge 1 commit into
Smartdevs17:mainfrom
fantababa:feature/add-skeleton-loading
Open

feat(ui): replace loading spinners with shimmer skeleton screens#513
fantababa wants to merge 1 commit into
Smartdevs17:mainfrom
fantababa:feature/add-skeleton-loading

Conversation

@fantababa
Copy link
Copy Markdown

Closes #67

Replace ActivityIndicator and blank loading states with smooth shimmer
skeleton screens to improve perceived performance on initial data load.

  • SkeletonLoader: rewrote with a linear shimmer animation using
    expo-linear-gradient and Animated translateX on the native driver,
    replacing the previous opacity pulse. Added StatsCardSkeleton and
    SubscriptionDetailSkeleton to match the real layouts exactly
  • HomeScreen: reads isLoading from useSubscriptionStore and renders
    StatsCardSkeleton + SubscriptionListSkeleton (4 cards) while data
    is fetching; FilterBar and header remain visible during load
  • SubscriptionDetailScreen: replaces ActivityIndicator with
    SubscriptionDetailSkeleton wrapped in SafeAreaView
  • All skeleton components use useThemeColors() so shimmer colors
    adapt to light and dark mode automatically

Pull Request Checklist

Quality Gates (All must pass before merge)

  • Lint: Code passes ESLint and Prettier checks
  • Type Check: TypeScript compilation succeeds
  • Tests: All tests pass
  • Build: Project builds successfully
  • Rust Format: Smart contract formatting is correct
  • Rust Clippy: Smart contract linting passes
  • Rust Tests: All smart contract tests pass
  • Rust Build: Smart contracts compile successfully

Additional Requirements

  • New code has appropriate TypeScript types
  • No hardcoded secrets or credentials
  • New features have corresponding tests
  • Documentation updated if needed

Reviewers

  • At least 1 approval required for merge
  • All CI checks must be green

This PR will not be mergeable until all quality gates pass.

@drips-wave
Copy link
Copy Markdown

drips-wave Bot commented Jun 1, 2026

@fantababa Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@Smartdevs17
Copy link
Copy Markdown
Owner

⚠️ Merge failed — check CI status.

🤖 Drips Wave Merge Agent

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.

🔧 Add skeleton loading screens

2 participants