CRREL/allover.js — pull request #30 by chambbj (2012-11-30 20:18:50+00:00)
Title: Refactoring allover.js slightly

@gadomski, this pull request is not as big as it may seem. Still, I'd like to talk through a couple of aspects.

First, I didn't see that you'd been working on the stylesheet and moving html to the javascript until I'd already done it also! So, you'll see that in here, but I can back it out and merge your changes when they are ready.

Second, there is a great deal of code, primarily in profile-box.js, that was unused, so it has been removed. That's actually how this branch started, it just got a little out of control.

I've also been working with a new approach to setting up the viewer, that may play well with your idea of incrementally adding data to the viewer. It follows an approach similar to three.js,

```javascript

```

I've also done some things, like create functions for the sidebar, keyboard shortcut, and select box generation. On the latter, you'll now see that the colorbrewer elevation palettes are grouped under the elevation optgroup, and everything else is just an option of the select. This allows us to go with only one drop-down, and we don't have to worry about enabling/disabling

Most of the noise here comes in the form of performing some code formatting. Don't know how you feel about coding style, but it's a bit easier on my eyes to insert some more whitespace, which is evident in this branch.

So I don't think this is ready to go as-is, but let's discuss. I could quite easily break out the modification to the coloration select and submit a pull request separately, for example.