KateIurasheva / KateIurasheva.github.io

0 stars 2 forks source link

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

Open Petergod opened 9 years ago

Petergod commented 9 years ago

Визуально - все отлично. Размещение файлов - корректно нужно картинки чтобы были в папке с картинками, файлы стилей в папке со стилями пример корневой папки: "css (папка), img (папка), index.html" Структура и расположения тегов - хорошо. Абзацы лучше использовать тегом <p> а не<div> и тегу <p> - задать отступы. Файл стилей css: Для body можно не задавать высоту и ширину , нужные параметры Вы задали уже в блоке "main" background: white; - можно не писать, он задается автоматически. Для того, чтобы макет был по центру можно задать margin: 0 auto; вместо margin-left: 455px; в классе "main" . Чтобы картинку сделать по центру , можно в документе HTML добавить : `

my photo

Тогда в css где img отступpadding-left: 192px;можно убрать. В целом макет получился! Мои советы: В Вашем случае для тегов р h2 , в файле css лучше всего задавать классы просто прописавр {стили }`h2{стили } , тут к классам обращаться не нужно .main p{ } Задавать отступы margin-top , margin-left , margin-right правильно, но можно все задать просто в одном margin. Почитайте немножко об margin i padding. Но визуально все хорошо, просто можно немножко оптимизировать стили.