search

Skyleen77 / next-nprogress-bar

NProgress integration on Next.js compatible with /app and /pages folders
https://next-nprogress-bar.vercel.app
315 stars 24 forks source link

fix(AppProgressBar): ignore href that starts `javascript:` #42

Closed magnnus closed 6 months ago

vercel[bot] commented 6 months ago

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

Name Status Preview Comments Updated (UTC)
next-nprogress-bar ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 11, 2024 6:31am
magnnus commented 6 months ago

Maybe we can also add a scope for document.querySelectorAll('a').

Scenario 1： We use Nginx SSI to include some micro-service module to my Next.js app, eg: common header / footer, these micro-service not only serve Next.js apps but alse tranditional Java-Web apps.

Skyleen77 commented 6 months ago

Thanks for this MR !

Do not hesitate to make an MR to add a scope for document.querySelectorAll('a').

I will study it carefully