Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Improve startup performance and user experience #86

Closed
emxsys opened this issue Feb 9, 2018 · 3 comments
Closed

Improve startup performance and user experience #86

emxsys opened this issue Feb 9, 2018 · 3 comments
Labels
enhancement
Milestone
Explorer 0.2.0

Comments

@emxsys
Copy link
Member

emxsys commented Feb 9, 2018

Refactor module dependencies to display the globe ASAP.

  • Consider a progress indicator
  • Consider displaying a tessellation layer until basic layers are loaded.
@emxsys emxsys added this to the Explorer v0.2.0 milestone Feb 9, 2018
emxsys added a commit that referenced this issue Feb 10, 2018
@emxsys
Removed unused references to model/Explorer.
2f7c75d 
- Re: issue #86.
emxsys added a commit that referenced this issue Feb 10, 2018
@emxsys
Removed unused reference to model/Explorer.
3291c8a 
- Re: issue #86.
emxsys added a commit that referenced this issue Feb 10, 2018
@emxsys
GlobeViewModel now takes an Explorer as a param instead of dependency.
cb408a3 
- Re: issue #86.
emxsys added a commit that referenced this issue Feb 10, 2018
@emxsys
Refactored Explorer to initialize Knockout Model/View/View Models.
34b6fe0 
- Moved code from main.js.
- Re: issue #86.
emxsys added a commit that referenced this issue Feb 10, 2018
@emxsys
Refactored main.js to improve startup performance and user experience.
db957e7 
- Removed many require dependencies.
- Initializes the WorldWindow with background layers to display with the Explorer loads.
- Makes a second require call to load the Explorer and the view model dependencies async.
- Re: issue #86.
emxsys added a commit that referenced this issue Feb 10, 2018
@emxsys
The Globe nows clears any backgound layers added at startup by main.js.
c549f8d 
- Re: issue #86.
emxsys added a commit that referenced this issue Feb 10, 2018
@emxsys
Updated Config startupAltitude to display distant globe.
8e6f377 
- Re: #86
emxsys added a commit that referenced this issue Feb 10, 2018
@emxsys
Updated main.js to display WorldWind globe using Config startup param…
8ea2eed 
…eters

- Re: #86
@emxsys
Copy link
Member Author

emxsys commented Feb 10, 2018

Improvements

The globe is now displayed early in the startup and is now using a configurable startup position.

  • Refactored the application entry point (main.js) to minimize the number of dependencies at startup.
  • Now displaying the WorldWind globe with a background layer ASAP.
  • Now using the configuration startup parameters for the initial WorldWind globe position.
  • The Explorer object is initialized asynchronously after the WorldWind globe is initialized.
  • The Explorer was refactored to perform the Knockout Model-View-View Model initialization.

@emxsys emxsys closed this as completed Feb 10, 2018
emxsys added a commit that referenced this issue Feb 10, 2018
@emxsys
Changed popular libraries to use CDN links for production.
fc9d5ec 
- Re #86.
@emxsys
Copy link
Member Author

emxsys commented Feb 11, 2018

Additional Improvements

Added support for slow connections.

  • Using CDNs for the popular libraries instead of my (slow) personal web hosting service.
  • Added an error handler to provide feedback/advice on timeout errors.
  • Increased the script loading timeout period from 7s to 15s to handle slow 3G connections.

emxsys added a commit that referenced this issue Feb 12, 2018
@emxsys
Added PaceJS progress counter.
ee78f95 
- Added pace 1.0.2 library.
- main.js starts the counter.
- index.html control the style.
- Re: #86
@emxsys
Copy link
Member Author

emxsys commented Feb 12, 2018

Another Improvement

Added a progress counter that displays in the upper right during startup.

emxsys added a commit that referenced this issue Feb 12, 2018
@emxsys
Added PaceJS 1.0.2 library and themes.
19e7c9e 
- Updated index.html pace stylesheet.
- Re: #86
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
Explorer 0.2.0
Development

No branches or pull requests
1 participant
@emxsys