{"action":"created","issue":{"url":"https://api.github.com/repos/Ada-C5/Static-Site/issues/22","repository_url":"https://api.github.com/repos/Ada-C5/Static-Site","labels_url":"https://api.github.com/repos/Ada-C5/Static-Site/issues/22/labels{/name}","comments_url":"https://api.github.com/repos/Ada-C5/Static-Site/issues/22/comments","events_url":"https://api.github.com/repos/Ada-C5/Static-Site/issues/22/events","html_url":"https://github.com/Ada-C5/Static-Site/pull/22","id":145740164,"number":22,"title":"Yordanos Dirar - Static Site","user":{"login":"yordanosd","id":15112557,"avatar_url":"https://avatars.githubusercontent.com/u/15112557?v=3","gravatar_id":"","url":"https://api.github.com/users/yordanosd","html_url":"https://github.com/yordanosd","followers_url":"https://api.github.com/users/yordanosd/followers","following_url":"https://api.github.com/users/yordanosd/following{/other_user}","gists_url":"https://api.github.com/users/yordanosd/gists{/gist_id}","starred_url":"https://api.github.com/users/yordanosd/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/yordanosd/subscriptions","organizations_url":"https://api.github.com/users/yordanosd/orgs","repos_url":"https://api.github.com/users/yordanosd/repos","events_url":"https://api.github.com/users/yordanosd/events{/privacy}","received_events_url":"https://api.github.com/users/yordanosd/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2016-04-04T16:47:53Z","updated_at":"2016-04-11T07:22:51Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/Ada-C5/Static-Site/pulls/22","html_url":"https://github.com/Ada-C5/Static-Site/pull/22","diff_url":"https://github.com/Ada-C5/Static-Site/pull/22.diff","patch_url":"https://github.com/Ada-C5/Static-Site/pull/22.patch"},"body":"'m not on social media but I included social media links for fun.\r\n\r\nIt's really messy. \r\n\r\nThanks =)"},"comment":{"url":"https://api.github.com/repos/Ada-C5/Static-Site/issues/comments/208197772","html_url":"https://github.com/Ada-C5/Static-Site/pull/22#issuecomment-208197772","issue_url":"https://api.github.com/repos/Ada-C5/Static-Site/issues/22","id":208197772,"user":{"login":"theutahkate","id":5475551,"avatar_url":"https://avatars.githubusercontent.com/u/5475551?v=3","gravatar_id":"","url":"https://api.github.com/users/theutahkate","html_url":"https://github.com/theutahkate","followers_url":"https://api.github.com/users/theutahkate/followers","following_url":"https://api.github.com/users/theutahkate/following{/other_user}","gists_url":"https://api.github.com/users/theutahkate/gists{/gist_id}","starred_url":"https://api.github.com/users/theutahkate/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/theutahkate/subscriptions","organizations_url":"https://api.github.com/users/theutahkate/orgs","repos_url":"https://api.github.com/users/theutahkate/repos","events_url":"https://api.github.com/users/theutahkate/events{/privacy}","received_events_url":"https://api.github.com/users/theutahkate/received_events","type":"User","site_admin":false},"created_at":"2016-04-11T07:22:51Z","updated_at":"2016-04-11T07:22:51Z","body":"You have a lot of great stuff going on here - good job! Here's some thoughts on what I like in your code, and some things to work on:\r\n\r\n- I like your use of background-image - you're using it appropriately (for largely decorative images), and you're using the shorthand syntax, which is great.\r\n- Tables can be really tempting to use for layout purposes, like you have in your hobby blog and portfolio, and they used to be used widely for that purpose. Currently, they are used only for tabular data. In situations like that, or with photo galleries, one common practice is to use an unordered list. You can then have each entry as a list item (which can include more than just an image or bit of text), and then make those `<li>`s into columns with css.\r\n- Nice job importing the social icons font. That's good practice.\r\n- You don't need the `charset` attribute when you're linking to stylesheets, since you have that covered in your `<meta>` tag.\r\n- Great use of rems and %. Don't forget you can use rems with font size as well.\r\n- Your nav/social icons could have all the pieces in the right places - can you think of ways to make your header navs a bit more semantic? \r\n- Overall, your code is really readable. Your css in particular, is nicely organized and commented."}}