Refs #334, #403, add more loading indicators to app

brucecrevensten commented 7 years ago

Changes here include:

Adds an initial "Loading" screen as soon as the app loads
After the initial data load from GeoNode, when the splash screen is displayed, a new area blocking the buttons is present until all the underlying data has been loaded.

This change is

cstephen commented 7 years ago

Thoughts:

The icon looks fantastic! Small nitpick here, but we could make this look a little better by reducing the width of the div so it covers less of the background image:

Instead of this:

We could have this:

I can add a commit for this change if you agree that this looks better, @bruce.

I noticed some console.log() statements in the code. Should these be removed?
I accidentally messed up the GEOSERVER_URL environment variable on my local instance and noticed it acts like everything loaded properly after all of my GeoServer requests failed. Looks like we have an opportunity to report network errors using this new div & infrastructure. But I consider this outside the scope of this PR, and we already have an issue (#302) open for this.

bruce commented 7 years ago

Hey folks! Love what you're doing, but you probably mean @brucecrevensten.

🍻

ua-snap / mapventure