apal21 / nextjs-progressbar

A simple Next.js progressbar component using NProgress.
https://www.npmjs.com/package/nextjs-progressbar
MIT License
786 stars 62 forks source link

Closes #17 #18

Closed anubra266 closed 3 years ago

anubra266 commented 3 years ago

It's often preferable to delay showing the loading indicator until a request has taken longer than 250ms-500ms. This prevents the loading indicator from appearing constantly on quick page visits, which can be visually distracting.

apal21 commented 3 years ago

Thanks for the PR! I'll review and merge this!

anubra266 commented 3 years ago

Happy to help.

anubra266 commented 3 years ago

Please, when will you review the PR. So we can make update it in our projects.

apal21 commented 3 years ago

This code is not working as intended. The progress bar is getting stuck on route change. I tested this code with this repo: https://github.com/apal21/demo-nextjs-progressbar