search

rodrigobastosv / loading_overlay

MIT License
122 stars 32 forks source link

Documentation for global overlay #45

Open Pingolus opened 1 year ago

Pingolus commented 1 year ago

Hello,

thanks for this plugin !

I have a suggestion for documentation about global overlay widget.

If your need is to use loading_overlay for an entire app loader, shown from context every time and from different pages you have to wrap your main with GlobalLoaderOverlay and not just LoaderOverlay.

I did not see this point in the documentation because paragraph about routing is about named routes and I don't have named routes in my app ; so my bad but I think the title "Basic Usage on Named Routes" on the documentation could be rewrite to "Basic Usage With Navigation" with or without named routes.

Thanks for your job !

rodrigobastosv commented 1 year ago

Hey @Pingolus , thank you so much for using this package and thanks you for the feedback. I will update the documentation soon with your suggestion.