Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added the Nextjs Loader #22

Closed
wants to merge 1 commit into from
Closed

Conversation

wenayy
Copy link

@wenayy wenayy commented Oct 8, 2024

Overview

This pull request integrates the nextjs-toploader library into the project to enhance the user experience during page navigation. The loading indicator will display at the top of the page while content is being fetched.

Changes Made

  • Installed nextjs-toploader library.
  • Imported and configured TopLoader in layout.jsx:
    • Set the showSpinner property to false to hide the spinner.
    • Customized the loading bar color to #29d.

Benefits

  • Provides a smoother visual feedback to users during page loads.
  • Improves user experience by indicating loading status without distracting spinners.

Testing

  • Navigated through various pages to ensure the top loader appears and functions as intended.
  • Verified that no spinner is displayed, and the loading bar color is as specified.

Copy link

vercel bot commented Oct 8, 2024

Someone is attempting to deploy a commit to the Chirag Aggarwal's projects Team on Vercel.

A member of the Team first needs to authorize it.

@wenayy wenayy mentioned this pull request Oct 8, 2024
Copy link

vercel bot commented Oct 9, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
cal-buddy ❌ Failed (Inspect) Oct 9, 2024 8:39am

@ChiragAgg5k
Copy link
Owner

@wenayy there are build errors. please check them.

Screenshot 2024-10-09 at 2 10 44 PM

@ChiragAgg5k
Copy link
Owner

PR Closed due to inactivity

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