Skip to content

feat: implement global loading and error states (#76)#77

Merged
akash-kumar-dev merged 1 commit intobrowseping:mainfrom
shruthideepikareddy:main
Mar 2, 2026
Merged

feat: implement global loading and error states (#76)#77
akash-kumar-dev merged 1 commit intobrowseping:mainfrom
shruthideepikareddy:main

Conversation

@shruthideepikareddy
Copy link
Copy Markdown
Contributor

@shruthideepikareddy shruthideepikareddy commented Mar 1, 2026

Description

This PR implements root-level loading and error handling states using Next.js built-in loading.tsx and error.tsx conventions. It significantly improves the "perceived performance" by providing a consistent, branded UI during navigation and a robust fallback for unexpected errors.

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Code refactoring
  • Performance improvement
  • UI/UX improvement
  • Content update
  • Chore/maintenance

Closes #76

Changes Made

Please list the specific changes made in this PR:

Global Loading State: Added src/app/loading.tsx featuring a premium animated background and "Syncing Data" indicator.

Global Error Boundary: Added src/app/error.tsxwith a recovery UI ("Try Again" and "Back to Home") and specific styling for error alerts.

Leaderboard Skeleton: Added a specialized LeaderboardSkeleton component and integrated it into the leaderboard route for a smoother data-fetching experience.

Testing Done

Please describe how you tested your changes:

  • Tested in development mode (npm run dev)
  • Tested production build (npm run build && npm start)
  • Tested on multiple browsers (list them)
  • Tested on mobile devices/responsive design
  • Ran linting (npm run lint)
  • Checked for console errors

Screenshots (if applicable)

Screenshot 2026-03-01 191817 Screenshot 2026-03-01 191725

Checklist

IMPORTANT: Please ensure all items are checked before submitting. PRs that don't follow these guidelines may be closed.

  • I have read and followed all guidelines in CONTRIBUTING.md
  • I have used the correct branch naming convention (feature/, fix/, docs/, etc.)
  • My commits follow Conventional Commits format (feat:, fix:, docs:, etc.)
  • I have performed a self-review and tested my changes thoroughly
  • CRITICAL: I confirm ONLY my intended changes are included (no unrelated changes, debug code, or breaking changes)
  • I have updated relevant documentation if needed
  • I have tested the production build locally
  • My changes are responsive and work on mobile devices

Additional Notes

Any additional information that would be helpful for reviewers?


Remember: Quality over quantity. A well-tested, properly documented small PR is better than a large, rushed contribution.

For more details, please refer to our Contributing Guidelines.

@vercel
Copy link
Copy Markdown

vercel bot commented Mar 1, 2026

@shruthideepikareddy is attempting to deploy a commit to the Akash Kumar's projects Team on Vercel.

A member of the Team first needs to authorize it.

@akash-kumar-dev akash-kumar-dev self-requested a review March 2, 2026 20:44
@vercel
Copy link
Copy Markdown

vercel bot commented Mar 2, 2026

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

Project Deployment Actions Updated (UTC)
web Ready Ready Preview, Comment Mar 2, 2026 8:45pm

@akash-kumar-dev
Copy link
Copy Markdown
Member

Everything looks good to merge. Thanks @shruthideepikareddy for your valuable contributions!

@akash-kumar-dev akash-kumar-dev merged commit d3b2d38 into browseping:main Mar 2, 2026
2 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.

Feature Request: Add Global Loading and Error States

2 participants