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

[VadimZhiltsov]

1) не закрытый тег

<div class="menu-reg">

2) Это надо сделать через тег legend, чтоб семантично дать имя форме

<h1>Cadastre-se<h1>

3) Использование hr тут не уместно. По стандарту W3C это больше не визуальный тег, а семантический. Он должен служить как семаничная смена темы

<hr noshade size="1">

4) Многие теги input не имеют закрывающегося знака >

5) В css слишком много дублирования кода, надо сгрупировать:

.first{
    height: 37px;
    width: 566px;
    margin-top: 10px;
    border-radius: 40px;

}

.last{
    height: 37px;
    width: 566px;
    margin-top: 10px;
} 

.email{
    height: 37px;
    width: 566px;
     margin-top: 10px;

}

.password{
    height: 37px;
    width: 566px;
     margin-top: 10px;

}

.confirm{
    height: 37px;
    width: 566px;
    margin-top: 10px;

}

.send{
    height: 37px;
    width: 539px;
    margin-top: 10px;
    font-size: 17px;
    margin-right: 97px;

}

6) Используются общие селекторы в css. Надо задавать больше конкретики т.к. такая реализация затрудняет поддержку приложения

7) Проблема с отступами в коде, что делает его не читаемым. Пункт обязателен к исправлению для зачёта.

[Yaroslav1984]

принято. Буду исправлять замечания

[Yaroslav1984]

1) не закрытый тег

- закрыл 2) Это надо сделать через тег legend, чтоб семантично дать имя форме - сделал 4) Многие теги input не имеют закрывающегося знака > - сделал 5) В css слишком много дублирования кода, надо сгрупировать - сделал 7) Проблема с отступами в коде, что делает его не читаемым. Пункт обязателен к исправлению для зачёта - исправил 6) Используются общие селекторы в css. Надо задавать больше конкретики т.к. такая реализация затрудняет поддержку приложения - я по убирал лишние лублирующие селекторы + те которые не работают 3) Использование hr тут не уместно. По стандарту W3C это больше не визуальный тег, а семантический. Он должен служить как семаничная смена темы - сделал

[VadimZhiltsov]

[16:52:31] Vadim Zhiltsov: со стороны кода там всё ок, просто тестировщик не поставил апрув, что там всё ок [16:52:53] Vadim Zhiltsov: т.е. когда он заапрувит, то я сразу поставлю зачёт

[VadimZhiltsov]

Зачёт