Проверка Д/3 Lesson 6

VadimZhiltsov commented 8 years ago

http://oi68.tinypic.com/o70l78.jpg

Очень большие расхождения с макетом, суть задачи pixel perfect, а это значит что все блоки, ярлычки, шрифты должны расходиться хотяб на несколько пикселей. То что не совпадают шрифты, можно забить, но позицию текста надо выравнять. Исправь их, потом будем смотреть код.

VadimZhiltsov commented 8 years ago

1) Нет такого тега

<h class="tell">(044)567-39-10</h>

2) Негоже называть класы транслитом.

3) Классы figure_1 figure_2 figure_3 не используются

4) Градиент сделать кроссбраузерным

5) Селекторы по аттрибутам работают значительно дольше, чем селекторы по классам. Надо поменять селекторы:

input[value="регистрация"], input[value="вход"], input[type="name"]{

Детали можно почитать: https://developer.mozilla.org/en-US/docs/Web/Guide/CSS/Writing_efficient_CSS

6) Закоментировать все коментарии к коду.

color: #2a4263; Цвет обычной ссылки

7) Задать больше конкретики для селектора. Этот код ограничивает использование чекбоксов

input[type="checkbox"]{

Yaroslav1984 commented 8 years ago

1) Нет такого тега (044)567-39-10 - исправил сделал sраn 2) Негоже называть класы транслитом - по максимуму исправил 3) Классы figure_1 figure_2 figure_3 не используются - уже используются 4) Градиент сделать кроссбраузерным - сделал 5) Селекторы по аттрибутам работают значительно дольше, чем селекторы по классам. Надо поменять селекторы: - исправил на input[type="name_login"]{, input[type="submit_entrance"]{, input[type="submit_check"]{, input[type="button_password"]{ 6) Закоментировать все коментарии к коду - сделал Задать больше конкретики для селектора. Этот код ограничивает использование чекбоксов input[type="checkbox"]{ - сделал

VadimZhiltsov commented 8 years ago

1)

.figure_one{
  height: 195px;
  width: 510px;
}

.figure_two{
  height: 198px;
  width: 510px;
}

.figure_three{
  height: 204px;
  width: 510px;
}

Один и тот же стиль повторяется 3 раза. Лучше сделать так:

.figure_one, .figure_two, .figure_three{
  width: 510px;
}

.figure_one{
  height: 195px;
}

.figure_two{
  height: 198px;
}

.figure_three{
  height: 204px;
}

2)

   border-left-width: 0px;
    border-top-width: 0px;
    border-bottom-width: 0px;
    border-right-width: 0px;

лучше объеденить это в одно свойство: border-width: 0px 0px 0px 0px

Yaroslav1984 commented 8 years ago

Исправления сделал

VadimZhiltsov commented 8 years ago

Зачёт

Yaroslav1984 / Yaroslav1984.github.io

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