{"comment":{"url":"https://api.github.com/repos/gennadybaranov/famci/comments/20943231","html_url":"https://github.com/gennadybaranov/famci/commit/a35aee565e86555934ecc4dba452f702f02f6855#commitcomment-20943231","id":20943231,"user":{"login":"gennadybaranov","id":20395980,"avatar_url":"https://avatars.githubusercontent.com/u/20395980?v=3","gravatar_id":"","url":"https://api.github.com/users/gennadybaranov","html_url":"https://github.com/gennadybaranov","followers_url":"https://api.github.com/users/gennadybaranov/followers","following_url":"https://api.github.com/users/gennadybaranov/following{/other_user}","gists_url":"https://api.github.com/users/gennadybaranov/gists{/gist_id}","starred_url":"https://api.github.com/users/gennadybaranov/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/gennadybaranov/subscriptions","organizations_url":"https://api.github.com/users/gennadybaranov/orgs","repos_url":"https://api.github.com/users/gennadybaranov/repos","events_url":"https://api.github.com/users/gennadybaranov/events{/privacy}","received_events_url":"https://api.github.com/users/gennadybaranov/received_events","type":"User","site_admin":false},"position":null,"line":null,"path":null,"commit_id":"a35aee565e86555934ecc4dba452f702f02f6855","created_at":"2017-02-18T05:42:54Z","updated_at":"2017-02-18T05:42:54Z","body":"Valeriya, please do the following changes:\r\n\r\n1. Don't use tables.\r\n2. Don't use inline styles. Create separate css file, move your styles there.\r\n3. Right now, I have to copy your images to the same folder with html page. Use relative references in HTML to your images to keep them in folder images.\r\n4. Don't use Russian names for images.\r\n5. For text boxes with specific content(like email) it's better to use correspondent type:  https://www.w3schools.com/html/html_form_input_types.asp\r\n6. Use placeholders for text boxes. \r\n7. Add hover effects for buttons, links.\r\n8. Remove empty spaces between content blocks.\r\n9. Add shadow to your content.\r\n10. Align header(menu) to the same size as your content.\r\n11. Make menu items look similar to mockup: vertical separator + hover, press states. "}}
{"actor":{"display_login":"gennadybaranov"}}