The latest version of Elipse claims ther are over 100 unread local variables in our code base, 0 of them appear to be accurate. Since the warning is now useless, we should ignore it.
1. Submit knockout 2.2.1 to Source/ThirdParty
2. Start refactoring Animation (and Cesium widgets in genereal) to use an MVVM architecture.
3. Cleanup Eclipse project to exclude JS validation on all Source/ThirdParty code.
Make the files match their upstream form as closely as possible. We should not be aggressively rewriting and reformatting our third-party libraries to match our own coding standards, because it makes it much harder to diff against the original when trying to upgrade.
Also adjust the licensing comments to:
* be correctly preserved in minified output by both r.js (UglifyJS) and Dojo (Closure Compiler)
* clearly indicate the names and URLs of the components each license applies to.
Replace Jobs.downloadImage and getImageFromUrl with promise-based loadImage.
Change jsonp to work in a promise-based way. Fix some other random documentation
problems I noticed along the way.
There were still some options that defaulted to the unexpected choice,
so I finally gave in and specified values for all the options on the webpage.
Also turn on browser globals across the board.
Scott Hunter
Matthew Amato
Patrick Cozzi
Ed Mackey