nprogress always show bar
See original GitHub issueI know that it isn’t generally needed to show the progress bar all the time. However, small mobile phones the size of the screen can be small enough that header content covers a relatively large portion of the screen.
While a keen observer may be able to notice a small portion of screen changing, it can be difficult, especially when pages update as quickly as they do when using Gatsby. The problem is expounded by not having any indicator from the browser showing that the action has been completed either.
For my use case I had a small bio at the top of the page on mobile. Without scrolling down you could click on a blog topic and be sent to the full version of the article. But the article also had the small bio at the top of the page and the same excerpt at the top so besides some spacing differences it was hard even for me to notice the difference. I solved my use case by moving the bio to the bottom because it isn’t extremely useful, especially on mobile. But I think the progress bar could be useful for clarity purposes.
Regardless, the Gatsby docs specify “you may pass all available nprogress configuration options.” to nprogress plugin. In Gatsby’s case it isn’t 100% accurate because passing the minimum
configuration option doesn’t change anything in Gatsby’s case.
TL;DR consider implementing support for the minimum
configuration option for nprogress instead of the set 1s delayed loading.
Issue Analytics
- State:
- Created 6 years ago
- Reactions:1
- Comments:7 (4 by maintainers)
Top GitHub Comments
Great idea @fk, even got a response from @KyleAMathews himself. Closing in favor of #3368
👋 @Sorgrum Should we close this in favor of #3368 [gatsby-plugin-nprogress]: change default show delay which is a little more recent/active?