{"action":"created","issue":{"url":"https://api.github.com/repos/loewen-uwaterloo/my-website/issues/1","labels_url":"https://api.github.com/repos/loewen-uwaterloo/my-website/issues/1/labels{/name}","comments_url":"https://api.github.com/repos/loewen-uwaterloo/my-website/issues/1/comments","events_url":"https://api.github.com/repos/loewen-uwaterloo/my-website/issues/1/events","html_url":"https://github.com/loewen-uwaterloo/my-website/issues/1","id":93915889,"number":1,"title":"Ben's thoughts","user":{"login":"DukeBaird","id":7780602,"avatar_url":"https://avatars.githubusercontent.com/u/7780602?v=3","gravatar_id":"","url":"https://api.github.com/users/DukeBaird","html_url":"https://github.com/DukeBaird","followers_url":"https://api.github.com/users/DukeBaird/followers","following_url":"https://api.github.com/users/DukeBaird/following{/other_user}","gists_url":"https://api.github.com/users/DukeBaird/gists{/gist_id}","starred_url":"https://api.github.com/users/DukeBaird/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/DukeBaird/subscriptions","organizations_url":"https://api.github.com/users/DukeBaird/orgs","repos_url":"https://api.github.com/users/DukeBaird/repos","events_url":"https://api.github.com/users/DukeBaird/events{/privacy}","received_events_url":"https://api.github.com/users/DukeBaird/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-07-08T23:55:58Z","updated_at":"2015-07-09T05:35:52Z","closed_at":"2015-07-09T05:35:52Z","body":"You're including the movie sorting js on every page - this seems excessive. if you dont use it, dont include it. If its truly a \"global\" js file, it shouldnt have single page specific js.\r\n\r\nIf you're loading the js at the end of the html file, you dont need $(document).ready either.\r\n\r\nYour sidebar flickers, you can chain your css classes together to have them default to hidden on load.\r\n\r\nYou have a bunch of console errors on the movie listings page - its looking for images that arent there.\r\nAlso maybe some labels on the sliders, now that I know theyre for sorting. While im thinking about it, typing in the boxes and pressing enter doesnt adjust the sliders, but moving the slider adjusts the box. The poor box must feel excluded.\r\n\r\nWhy is your error log in your git repo?\r\n\r\nYou have a css file at the top of your project?\r\n\r\nYou have a lot of JQuery. Personally not a fan, but I understand the appeal.\r\n\r\nNice work on the templating, it looks good."},"comment":{"url":"https://api.github.com/repos/loewen-uwaterloo/my-website/issues/comments/119824925","html_url":"https://github.com/loewen-uwaterloo/my-website/issues/1#issuecomment-119824925","issue_url":"https://api.github.com/repos/loewen-uwaterloo/my-website/issues/1","id":119824925,"user":{"login":"loewen-uwaterloo","id":3021886,"avatar_url":"https://avatars.githubusercontent.com/u/3021886?v=3","gravatar_id":"","url":"https://api.github.com/users/loewen-uwaterloo","html_url":"https://github.com/loewen-uwaterloo","followers_url":"https://api.github.com/users/loewen-uwaterloo/followers","following_url":"https://api.github.com/users/loewen-uwaterloo/following{/other_user}","gists_url":"https://api.github.com/users/loewen-uwaterloo/gists{/gist_id}","starred_url":"https://api.github.com/users/loewen-uwaterloo/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/loewen-uwaterloo/subscriptions","organizations_url":"https://api.github.com/users/loewen-uwaterloo/orgs","repos_url":"https://api.github.com/users/loewen-uwaterloo/repos","events_url":"https://api.github.com/users/loewen-uwaterloo/events{/privacy}","received_events_url":"https://api.github.com/users/loewen-uwaterloo/received_events","type":"User","site_admin":false},"created_at":"2015-07-09T05:35:52Z","updated_at":"2015-07-09T05:35:52Z","body":"commits\r\n0331a4ff805729a474edf797340aaf4428691318\r\n9fd53792fe0effd2fb7bc7e3a1a65122f541a8b2\r\nDoesn't fix sliders.  Will create separate issue"}}