Комментарий от ментора

[Petergod]

Визуально - все хорошо. Размещение файлов - корректно нужно картинки чтобы были в папке с картинками, файлы стилей в папке со стилями пример корневой папки: "css (папка), img (папка), index.html" Разметка html кода файла- homework1.html Можно было обойтись, Одним классом <div class="workspace"> для всей странички, других, можно было не задавать и одним классом для картинки и все. Не вижу в документе ни одного <hr> . Абзацы лучше использовать тегом <p> а не <div> и тегу <p> - задать отступы <br/>.. ul в p не берется! CSS -В body margin: 0 auto; здесь не нужно. -В body можно было написать размер шрифта font-size; и добавить бегрануд background-color: #eeeeee; -класс .header . убрать у Вас есть h1 и в нем нужно задать : h1{ text-align: center; font-size: 12px; }

• Все boder лучше убрать а в HTML поставить <hr>
• .foto { text-align: center; height: 150px; width: 150px; } Должно быть так. margin ненужно!
• .lifestyle , .reason, .phone , .profile, .feedback - Все убрать.
• Задайте ширину и отступы в классе workspace примерно : width: 460px; padding: 30px 50px; не нужно было задавать параметр height: 806px; так как если текста будет больше, документ не потянется или задать параметр min-height. Сюда же добавьте margin: 0 auto; background-color: #ffffff; border: 1px #d5d5d5; box-shadow: 0 0 10px #d5d5d5; -В .whitepaper position: center; неправильно,такого нет, какой то новый стиль=) Весь класс можно удалить так,как все что нужно сделать - сделали в классе workspace.

Жду изменений!

[Petergod]

Размещение файлов - хорошо! Разметка html кода файла

• Как говорилось на вебинаре: тег <br> лучше не использовать, взять в span или другой тег и задать display:block Пример: <span>текст для переноса</span> и в CSS span: {display:blok}
• <hr> нужно убрать, линии нужно сделать через стили CSS
• Для картинки <center> </center> убираем и берем в блок, <div class="photo"> <img src="images/for_web_BML9121.jpg" alt="Изображение пользователя"> </div> и центрировать: text-align: center;
• Все остальное хорошо!

CSS

• Так, как мы будем уже работать с CSS (добавлять span, photo, и для линии) Рекомендую: .workspace ширину уменьшить на 400px , картинку уменьшить высоту и ширину img { height: 150px; width: 150px; }
• для р , вместо padding ,делаем margin-left:10px; . Код : p, ul{ margin-left:10px; }

[Petergod]

Разметка html кода файла

• Как говорилось на вебинаре: тег <br> лучше не использовать, взять в span или другой тег и задать display:block Пример: <p>Мой контактный телефон<span>+380677326468</span></p> и в CSS span: {display:blok}
• для линий лучше задавать отдельный блок , чтобы Вы могли ими управлять независимо от параграфа р . Пример: `

  Ирина Ляшенко

  

  В поиске себя

  `
• Все остальное хорошо.

CSS

• Когда правильно зададим линии , у нас будут большие отступы , чтобы их не было - нужно удалить лишние стили . из ul удалить padding , из р также.

Рекомендация

• так, как у р и ul одинаковые стили ,можем их объединить p,ul { line-height: 20px; margin-left: 10px; }
• также фото и h1 .foto,h1{ text-align: center; font-size: 20px; }

Исправите,дайте знать!!!

[Petergod]

ЗАЧЕТ