search

Yaroslav1984 / Yaroslav1984.github.io

0 stars 0 forks source link

Проверка Д/3 Lesson 1 #1

Open VadimZhiltsov opened 9 years ago

VadimZhiltsov commented 9 years ago

General:

  1. Следует придерживаться общего стиля кода: http://habrahabr.ru/post/143452/ Сейчас из-за не ровных отступов код читается с трудом.
  2. Страница не соответствует требованиям. Посмотрите на макет, который надо было сверстать https://github.com/goit-fe/markup_fe2o/blob/master/lesson_01/homework1.png Страница выглядит как лист А4.

CSS:

  1. margin: -9px 10px -11px 138px; Картинку лучше автоматически отцентрировать, а не руками попиксельно. При изменении размера картинки прийдётся опять её вручную центрировать, что усложняет поддержку такого кода.
  2. Ссылки не должны сливаться с текстом. Они должны быть чётко выражены. По номеру телефона не понятно, что на него можно кликать. Да и в тексте ссылки, наверно, лучше оставить только номер телефона.

HTML: 1) Нельзя использовать аттрибут align для html элементов, он считается устаревшим и может быть удалён из новых версий браузеров. Чтоб не возникло такого казуса, лучше использовать css align. Ссылка на HTML спецификацию: http://www.w3.org/TR/html-markup/p.html

The align attribute on the p element is obsolete. Use CSS instead.

2) Контактные данные семантично верно оборачивать тегом adress. (Это не обязательный пункт) 3) Один и тотже css фаил подключается дважды. 4) Пропущено равно в аттрибуте alt на картинке

 alt"Это мое фото">

5) Первая часть тега div не закрыта. Не закрыт тег 

<div class="profile"
  <p> Мой профиль в Facebook:
  <a href="//www.facebook.com/yaroslav.zamorsky"target="_blank"><br>facebook.com</a>
</div>

6) Первая часть тега html не закрыта. 

<html lang="en"
Yaroslav1984 commented 9 years ago

General: 1-й пункт - все ок CSS:

  1. margin: -9px 10px -11px 138px; - все ок HTML: 4 пункт alt"Это мое фото"> - все сделал корректно 5 пункт - сделал
Yaroslav1984 commented 9 years ago

п6 - сделал

Yaroslav1984 commented 9 years ago

путь папки и файла css поменял

Yaroslav1984 commented 8 years ago

На формат А4 выведено

VadimZhiltsov commented 8 years ago
  1. Ни один класс из не html не используется.
  2. В CSS используются только теговые селекторы, что ухудшает маштабируемость страницы, надо давать какие-то уточнения, ведь добавить новые элементы на страницу будет сложнее из-за общих селекторов. Лучше прописать стили через классы.
  3. В css есть селектор на тег tel, но в спецификации HTML такого тега нет.
  4. Выравнять отступы.
  5. Убрать аттрибут align он считается устаревшим и его поддержка прекращена
VadimZhiltsov commented 8 years ago

Зачёт. Тольк в следующий раз таки надо отформатировать отступы

Yaroslav1984 commented 8 years ago

спасибо