nickstark/project5115 — Issue #3: Some Notes
Author: nickstark
Created: 2011-12-07 23:28:45+00:00
State: open

Okay, I've noticed a few things going on...

### Filtering
First, if you want to change anything on the character builder side of the screen you **MUST** filter by the active tab first, otherwise you'll fuck up the others. For example. If you want to get the level selector you cannot write this:

document.getElementsByClassName('.level-picker')[0];

You have to filter first:

document.getElementsByClassName('.active-tab')[0].getElementsByClassName('.level-picker')[0];

### Class Names
Two things about class names. One: **Never** use an id if there's even the smallest chance that element will be used more than once on the page, use a class instead. It won't work. Basic javascript. I know this adds like 8 keystrokes but c'mon. Two: Do not change the class names already on the page, just add another. It may seem like a better name or whatever but someone else's code probably depends on it being called what it's called. So just add a second class, or third, or twentieth.

### Working to Working
Please don't push any broken code to github. It makes it hard for everyone to work on other stuff. And it makes this kitty sad :(

![sad cat](http://i181.photobucket.com/albums/x155/mystumbles/sad-cat.jpg)

Now let's finish this sonofabitch.