{"action":"created","issue":{"url":"https://api.github.com/repos/Vovchik228/Vovchik228.github.io/issues/2","labels_url":"https://api.github.com/repos/Vovchik228/Vovchik228.github.io/issues/2/labels{/name}","comments_url":"https://api.github.com/repos/Vovchik228/Vovchik228.github.io/issues/2/comments","events_url":"https://api.github.com/repos/Vovchik228/Vovchik228.github.io/issues/2/events","html_url":"https://github.com/Vovchik228/Vovchik228.github.io/issues/2","id":114607999,"number":2,"title":"Комментарий от ментора ДЗ №3","user":{"login":"Petergod","id":15101924,"avatar_url":"https://avatars.githubusercontent.com/u/15101924?v=3","gravatar_id":"","url":"https://api.github.com/users/Petergod","html_url":"https://github.com/Petergod","followers_url":"https://api.github.com/users/Petergod/followers","following_url":"https://api.github.com/users/Petergod/following{/other_user}","gists_url":"https://api.github.com/users/Petergod/gists{/gist_id}","starred_url":"https://api.github.com/users/Petergod/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Petergod/subscriptions","organizations_url":"https://api.github.com/users/Petergod/orgs","repos_url":"https://api.github.com/users/Petergod/repos","events_url":"https://api.github.com/users/Petergod/events{/privacy}","received_events_url":"https://api.github.com/users/Petergod/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-11-02T14:46:00Z","updated_at":"2015-11-03T20:14:42Z","closed_at":null,"body":"- скачет верхнее меню я об этом говорил в чате, что бы не скакали менюшки при наведении\r\nпоявляются бордеры, и скачет,\r\nкак вариант задаете margin: 0 1px, при ховере margin: 0; + бордеры). \r\n- нижнее меню не работает при наведении \r\n- картинка с велосипед не ровно стоит, \r\n- Лучше всего пользуйтесь clearfix \r\n- В Html Вы каждый элемент Вы взяли в отдельный блок, это нелогично) попробуйте например в один блок поместить картинку лого и менюшку,  левое меню картинка с велосипедом  текст также в отдельный общий блок, ну и футтер  копирайтинг.( Получается 1 общий блок например header внутри блока , блок с картинкой лого и верхней менюшкой ) . Так будет легче.И пользуйтесь clearfix\r\n- height: min-heigt;  новшество какое то))\r\n- Для велосипеда , марджин уберите, он нужен разве что для отступа от нижнего текста ито ,марджин боттом,\r\n- жирные линии в менюшке слева. \r\n- для текста ненужно display:inline-block; , padding-left:330px охх  много) поэтому и говорю ,чтобы помещать в один общий блок, это получается будет большой контейнер, а внутри него еще маленькие контейнеры которые нужно правильно разместить) так легче будет.\r\n- про копирайт услышали - супер! не работает менюшка при наведении \r\n\r\nПоиграйтесь,жду исправлений! \r\n"},"comment":{"url":"https://api.github.com/repos/Vovchik228/Vovchik228.github.io/issues/comments/153474072","html_url":"https://github.com/Vovchik228/Vovchik228.github.io/issues/2#issuecomment-153474072","issue_url":"https://api.github.com/repos/Vovchik228/Vovchik228.github.io/issues/2","id":153474072,"user":{"login":"Petergod","id":15101924,"avatar_url":"https://avatars.githubusercontent.com/u/15101924?v=3","gravatar_id":"","url":"https://api.github.com/users/Petergod","html_url":"https://github.com/Petergod","followers_url":"https://api.github.com/users/Petergod/followers","following_url":"https://api.github.com/users/Petergod/following{/other_user}","gists_url":"https://api.github.com/users/Petergod/gists{/gist_id}","starred_url":"https://api.github.com/users/Petergod/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Petergod/subscriptions","organizations_url":"https://api.github.com/users/Petergod/orgs","repos_url":"https://api.github.com/users/Petergod/repos","events_url":"https://api.github.com/users/Petergod/events{/privacy}","received_events_url":"https://api.github.com/users/Petergod/received_events","type":"User","site_admin":false},"created_at":"2015-11-03T20:14:42Z","updated_at":"2015-11-03T20:14:42Z","body":"- Проверьте на валидность HTML код )\r\n- вместо pre лучше используйте р и задайте марджини \r\n- div нужно назвать.\r\n- К clearfix задайте еще один класс,картинку переместите под левое меню, картинку и параграф возьмите в общий блок . К футеру также добавьте класс, копирайт под список. Примерно вот: http://joxi.ru/RmzzvnRfqR0amO\r\n\r\n**CSS** \r\n- для field рекомендую сделать меньшую ширину и сделать одинаковые паддинги код : `.field{\r\n\twidth: 950px;\r\n\tmargin: 0 auto;\r\n\tpadding: 50px;\r\n\tbackground-color: white;\r\n\tbox-shadow: 0 5px 5px rgba(0,0,0,0.3);\r\n\tfont: 12pt sans-serif;\r\n}`\r\n- для лого уберите марджины оставьте только `line-height: 0;`\r\n- для верхнего меню рекомендую параметры http://joxi.ru/bmovB5DFa3VRry\r\n- При наведении на левый список он скачет, жирные линии , их не должно быть .\r\n чтобы их не было нужно правильно задать бордеры , бордер должен быть 1px. \r\nКод Меню : `.sidebar{\r\n\tborder: solid 1px #c8c8c8;\r\n\tfloat: left;\r\n}\r\n.sidebar li {border-bottom: solid 1px #c8c8c8;}\r\n.sidebar li:last-child {\r\n\tborder-bottom: none;\r\n}\r\n.sidebar li a{\r\n\tdisplay: block;\r\n\twidth: 214px;\r\n\tpadding: 0 0 0 23px;\r\n\tline-height: 39px;\r\n\ttext-decoration: none;\r\n\tcolor: #656363;\r\n}\r\n.sidebar li:hover{\r\n\twidth: 212px;\r\n\tpadding: 0 0 0 25px;\r\n\tbackground-color: #f9f9f9;\r\n\tcolor: #2c2c2c;\r\n}\r\n.clearfix:after{\r\n\tcontent:'';\r\n\tclear: both;\r\n\tdisplay:block;\r\n}`  http://joxi.ru/5mdQGylFj3Nl21  Разберитесь что к чему )\r\n- Для нашего нового класса (контейнера) нужно задать падинги `.conteiner {padding: 40px 0 20px;}`\r\n- Для класса контента  нужно задать ширину, позицию текста,и флоат , тогда все станет на свои места.\r\n `.content {\r\n      float: right;\r\n\twidth: 627px;\r\n\ttext-align: left;}`\r\n- для Lorem не нужно паддингов, ширины, флоата,и так, как он у нас в р имеет свой класс , и находится внутри блока content (которому мы уже задали floar:right)  достаточно применить  `.Lorem {\r\n\tmargin:20px 15px;\r\n\tline-height: 1.5em;\r\n\tfont-size: 14px;\r\n\tcolor: #2c2c2c;\r\n} `\r\n- для картинки нужно сделать меньше высоту чуточку прибавить паддинг, флоат и марджин можно убрать, позиционировать картинку по центру можно свойством text-align: center\r\nКод: `.img{\r\n\theight: 340px;\r\n\tpadding:35px;\r\n\tbackground-color: white;\r\n\tborder: solid 1px #c8c8c8;\r\n\ttext-align: center;\r\n\t\r\n}`    http://joxi.ru/LmGj7bYhQJRdrl\r\n- Для bottom  margin-bottom лишнее , добавьте высоту и паддинг код : `.bottom {\r\n\theight: 40px;\r\n\tmargin-top:40px;\r\n\tpadding: 0 24px;\r\n\tbackground-color: #e1e1e1;\r\n}`\r\n- Для списка bottom li нужно float: left; и марджин margin: 0 40px;  `.bottom li {\r\n\tfloat: left;\r\n\tmargin: 0 40px;\r\n}`\r\n- Нужно убрать отступ первого элемента списка `.bottom li:first-child {\r\n\tmargin-left: 0;\r\n}`\r\n- Для бодотом и копирайта достаточно задать `.bottom li a, .copyright {\r\n\tdisplay: block;\r\n\tline-height: 40px; \r\n\ttext-decoration: none;\r\n\tcolor: #2c2c2c;\r\n}`\r\n- ну и отдельно для копирайта задать флоат \r\n    http://joxi.ru/12MpVbLSp04EmJ\r\n\r\nЖду исправлений!!!"}}