{"issue":{"state":"open","created_at":"2011-12-07T23:28:45Z","updated_at":"2011-12-07T23:28:45Z","user":{"gravatar_id":"dbbf4479c6a1c5f9b5498921babb9985","id":822325,"url":"https://api.github.com/users/nickstark","login":"nickstark","avatar_url":"https://secure.gravatar.com/avatar/dbbf4479c6a1c5f9b5498921babb9985?d=https://a248.e.akamai.net/assets.github.com%2Fimages%2Fgravatars%2Fgravatar-140.png"},"comments":0,"body":"Okay, I've noticed a few things going on...\r\n\r\n### Filtering\r\nFirst, 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:\r\n\r\n    document.getElementsByClassName('.level-picker')[0];\r\n\r\nYou have to filter first:\r\n\r\n    document.getElementsByClassName('.active-tab')[0].getElementsByClassName('.level-picker')[0];\r\n\r\n### Class Names\r\nTwo 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. \r\n\r\n### Working to Working\r\nPlease 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 :(\r\n\r\n![sad cat](http://i181.photobucket.com/albums/x155/mystumbles/sad-cat.jpg)\r\n\r\nNow let's finish this sonofabitch.","assignee":null,"id":2481738,"number":3,"labels":[],"pull_request":{"patch_url":null,"html_url":null,"diff_url":null},"title":"Some Notes","html_url":"https://github.com/nickstark/project5115/issues/3","closed_at":null,"url":"https://api.github.com/repos/nickstark/project5115/issues/3","milestone":null},"action":"opened"}