apal21 / nextjs-progressbar

A simple Next.js progressbar component using NProgress.

Home Page:https://www.npmjs.com/package/nextjs-progressbar

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

The `showOnShallow` option gets ignored during shallow routing

mschaul opened this issue · comments

The issue I ran into is that using shallow routing causes the progressbar to appear even with the showOnShallow option set to false. The main problem is that Next.js 12.2 fires the routeChangeError event unnecessarily. I have already reported it on their repo, but this also surfaced an issue with this component. The routeChangeError has a different function signature than routeChangeComplete, meaning the shallow routing flag gets ignored and the progressbar gets display even with showOnShallow set to false.

I found a previous mention of this issue here: #70 (comment), but it looks like it has not been fixed yet.

@apal21 can you add new version in npm rep?

I just check it in v0.0.16
It works now!
Thanks