{"action":"created","issue":{"url":"https://api.github.com/repos/annie0506/annie0506.github.io/issues/1","labels_url":"https://api.github.com/repos/annie0506/annie0506.github.io/issues/1/labels{/name}","comments_url":"https://api.github.com/repos/annie0506/annie0506.github.io/issues/1/comments","events_url":"https://api.github.com/repos/annie0506/annie0506.github.io/issues/1/events","html_url":"https://github.com/annie0506/annie0506.github.io/issues/1","id":113705051,"number":1,"title":"Комментарий от ментора","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-10-27T22:41:32Z","updated_at":"2015-10-28T08:35:18Z","closed_at":null,"body":"**Визуально** - не похожий на заданный макет, неправильно размещение макета, нет тени.\r\n**Размещение папок и файлов** - хорошо!\r\n**HTML**\r\n- Первый div нужно назвать либо class либо id, например `<div id=\"content\">`\r\n- Картинку нужно взять в отдельный блок : `<div class=\"photo\"> <img  src=\"img/fox1234.jpg\" alt=\"foxes\"> </div>` , ширину и высоту задать через CSS\r\n- Лишний span, лучше, вместо span задайте div и назовите его например \"line\" Код  `<div class=\"content\">`  (класс line понадобится для наших линий) \r\n- Как говорилось на вебинаре: тег `<br>` лучше не использовать, взять в span или другой тег и задать `display:block` Пример: `<p>Мой контактный телефон<span>+38066432**62</span></p>` и в CSS `span: {display:blok}`\r\n- `<hr>` нужно убрать, линии нужно сделать через стили CSS\r\n- От `\"Хочу учить Frontend потому что:...по...конец самого списка\"` взять в один блок назвать также \"line\" -  Код  `<div class=\"content\">` \r\n- третий блок  `\"Мой контактный тел.....по....последнюю ссылку\"` также взять в один блок и  назвать также line -  Код  `<div class=\"content\">` Также нужно взять в один параграф р\r\n- мой фидбек - все также нужно взять в один параграф р , и незабываем о переносах span\r\n**CSS**\r\n- body - если Вы хотели обнулить стили, тогда нужно задать `body {margin:0;\r\npadding:0;}` , Если Вы хотели разместить макет по центру , лучше свойство `margin:0 auto;` задать в .content , все остальные margin там удаляем. Для body задаем фон, и размер шрифтов  ` background-color: #eeeeee; font-size: 16px;`  Весь код будет: `body {margin:0;\r\npadding:0;\r\nfont-size: 16px;\r\nbackground:  #eeeeee;}`\r\n- .content - padding, border можно сократить записи) записав `padding: 40px 35px;\r\nborder:1px solid gray;` text-align удалить. Не нужно было задавать параметр height: 700px; так как если текста будет больше, документ не потянется или задать параметр min-height. И добавим тень  `box-shadow: 0 0 3px rgba(0, 0, 0, 0.3);` Код: `\twidth: 400px;\r\n\tmargin:5px auto;\r\npadding: 40px 35px;\r\nbackground:#fff;\r\nborder:1px solid gray;\r\n\t box-shadow: 0 0 3px rgba(0, 0, 0, 0.3);  `\r\n- Картинку вместе с h2 нужно будет поместить по центру, мы для картинки сделали отдельный блок и задаем `.photo,h2 {text-align:center;}` ну и зададим высоту фото `.photo img {height:140px;\r\n}`\r\n- Для линий с нашим классом \"line\" задаем стили  `.line {border-bottom:1px solid gray;}`\r\n- Незабываем о переносе строк - span , `span {display:block;}`\r\n- ну и зададим отступы параграфов и списка от левого края `p,ul {margin-left:10px;}`\r\n- Если Вы хотите чтобы слово \"Привет!\" было отдельно по центру, возьмите задайте ему класс  \r\nв HTML  `<p class=\"hi\">Привет!</p> `, и в css задайте `.hi {text-align:center;}` также можно задать и другие стили.Но это не обязательно.Это Ваша креативность и желание!\r\n\r\nЖду исправлений! Помните, правильный и чистый код нужен не нам, а Вам. Как научитесь сейчас ,так и будет потом, но потом будет тяжелее =))"},"comment":{"url":"https://api.github.com/repos/annie0506/annie0506.github.io/issues/comments/151765261","html_url":"https://github.com/annie0506/annie0506.github.io/issues/1#issuecomment-151765261","issue_url":"https://api.github.com/repos/annie0506/annie0506.github.io/issues/1","id":151765261,"user":{"login":"annie0506","id":15038792,"avatar_url":"https://avatars.githubusercontent.com/u/15038792?v=3","gravatar_id":"","url":"https://api.github.com/users/annie0506","html_url":"https://github.com/annie0506","followers_url":"https://api.github.com/users/annie0506/followers","following_url":"https://api.github.com/users/annie0506/following{/other_user}","gists_url":"https://api.github.com/users/annie0506/gists{/gist_id}","starred_url":"https://api.github.com/users/annie0506/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/annie0506/subscriptions","organizations_url":"https://api.github.com/users/annie0506/orgs","repos_url":"https://api.github.com/users/annie0506/repos","events_url":"https://api.github.com/users/annie0506/events{/privacy}","received_events_url":"https://api.github.com/users/annie0506/received_events","type":"User","site_admin":false},"created_at":"2015-10-28T08:35:18Z","updated_at":"2015-10-28T08:35:18Z","body":"Спасибо Вам за протраченное время) Обнуление стилей - 3й урок (а это\r\nзадание - 1 урок), тоже касается тени (на сколько я помню речь о ткни\r\nпроскакивала на вебинаре в вопросах к 3й домашке), а про min-height как-то\r\nвообще не в курсе, про background в скайпе мы говорили написать или white\r\nили , Вы подсказали, background :#fff. Похоже , я сначала не поняла , что\r\nэти курсы не для новичков :)\r\n\r\n28 октября 2015 г., 0:41 пользователь Petergod <notifications@github.com>\r\nнаписал:\r\n\r\n> *Визуально* - не похожий на заданный макет, неправильно размещение\r\n> макета, нет тени.\r\n> *Размещение папок и файлов* - хорошо!\r\n> *HTML*\r\n>\r\n>    - Первый div нужно назвать либо class либо id, например <div\r\n>    id=\"content\">\r\n>    - Картинку нужно взять в отдельный блок : <div class=\"photo\"> <img\r\n>    src=\"img/fox1234.jpg\" alt=\"foxes\"> </div> , ширину и высоту задать\r\n>    через CSS\r\n>    - Лишний span, лучше, вместо span задайте div и назовите его например\r\n>    \"line\" Код <div class=\"content\"> (класс line понадобится для наших\r\n>    линий)\r\n>    - Как говорилось на вебинаре: тег <br> лучше не использовать, взять в\r\n>    span или другой тег и задать display:block Пример: <p>Мой контактный\r\n>    телефон<span>+38066432**62</span></p> и в CSS span: {display:blok}\r\n>    - <hr> нужно убрать, линии нужно сделать через стили CSS\r\n>    - От \"Хочу учить Frontend потому что:...по...конец самого списка\"\r\n>    взять в один блок назвать также \"line\" - Код <div class=\"content\">\r\n>    - третий блок \"Мой контактный тел.....по....последнюю ссылку\" также\r\n>    взять в один блок и назвать также line - Код <div class=\"content\">\r\n>    Также нужно взять в один параграф р\r\n>    - мой фидбек - все также нужно взять в один параграф р , и незабываем\r\n>    о переносах span *CSS*\r\n>    - body - если Вы хотели обнулить стили, тогда нужно задать body\r\n>    {margin:0; padding:0;} , Если Вы хотели разместить макет по центру ,\r\n>    лучше свойство margin:0 auto; задать в .content , все остальные margin\r\n>    там удаляем. Для body задаем фон, и размер шрифтов background-color:\r\n>    #eeeeee; font-size: 16px; Весь код будет: body {margin:0; padding:0;\r\n>    font-size: 16px; background: #eeeeee;}\r\n>    - .content - padding, border можно сократить записи) записав padding:\r\n>    40px 35px; border:1px solid gray; text-align удалить. Не нужно было\r\n>    задавать параметр height: 700px; так как если текста будет больше, документ\r\n>    не потянется или задать параметр min-height. И добавим тень box-shadow:\r\n>    0 0 3px rgba(0, 0, 0, 0.3); Код: width: 400px; margin:5px auto;\r\n>    padding: 40px 35px; background:#fff; border:1px solid gray; box-shadow: 0 0\r\n>    3px rgba(0, 0, 0, 0.3);\r\n>    - Картинку вместе с h2 нужно будет поместить по центру, мы для\r\n>    картинки сделали отдельный блок и задаем .photo,h2 {text-align:center;}\r\n>    ну и зададим высоту фото .photo img {height:140px; }\r\n>    - Для линий с нашим классом \"line\" задаем стили .line\r\n>    {border-bottom:1px solid gray;}\r\n>    - Незабываем о переносе строк - span , span {display:block;}\r\n>    - ну и зададим отступы параграфов и списка от левого края p,ul\r\n>    {margin-left:10px;}\r\n>    - Если Вы хотите чтобы слово \"Привет!\" было отдельно по центру,\r\n>    возьмите задайте ему класс\r\n>    в HTML <p class=\"hi\">Привет!</p>, и в css задайте .hi\r\n>    {text-align:center;} также можно задать и другие стили.Но это не\r\n>    обязательно.Это Ваша креативность и желание!\r\n>\r\n> Жду исправлений! Помните, правильный и чистый код нужен не нам, а Вам. Как\r\n> научитесь сейчас ,так и будет потом. =))\r\n>\r\n> —\r\n> Reply to this email directly or view it on GitHub\r\n> <https://github.com/annie0506/annie0506.github.io/issues/1>.\r\n>\r\n"}}