Repair-Project

My first landing page

ruslan660.github.io's People

Contributors

Watchers

ruslan660.github.io's Issues

Comments

Ruslan660.github.io/src/slider.js

Lines 64 to 66 in 2e36d13

if (menu_point[index].classList.contains('menu-point--active')){

}

Пустое условие не нужно
Ruslan660.github.io/src/slider.js

Line 41 in 2e36d13

function slider(index) {

Имя функции должно говорить о том, что эта функция делает

Comments

Не нужно подсвечивать первый пункт главного меню.
Ruslan660.github.io/index.html

Line 103 in ce48076

<h4 class="gallery__title active__title">Rostov-on-Don, Admiral</h4>

Это должны быть ссылки.
Ruslan660.github.io/index.html

Line 99 in ce48076

<img src="src/img/desc-switches.svg" alt="switches" class="description__switches">

Это должны быть пять элементов. Две стрелочки и три кружочка
Ruslan660.github.io/style.css

Line 20 in ce48076

h1, h2, h3, h4, h5, h6 {

Добавьте в эти стили font-weight: normal
Ruslan660.github.io/index.html

Line 177 in ce48076

<div class="btn info__button">from 900 rub / m<sup>2</sup></div>

Здесь нужно убрать поведение на :hover
Ruslan660.github.io/index.html

Line 226 in ce48076

<div class="feature__number"><h2>01</h5></div>

<h2> - это очень весомый тег в верстке. Не стоит его здесь использовать. Это просто декоративный номер. Используйте Heading для заголовков
Ruslan660.github.io/index.html

Line 264 in ce48076

<input type="checkbox" id="agree_checkbox" name="agree_checkbox" value="Agree">

Ruslan660.github.io/index.html

Line 141 in ce48076

<input type="checkbox" id="agree_checkbox" name="agree_checkbox" value="Agree">

Ruslan660.github.io/index.html

Line 423 in ce48076

<input type="checkbox" id="agree_checkbox--question" name="agree_checkbox" value="Agree">

id на странице должен быть уникальным
Ruslan660.github.io/style.css

Line 995 in ce48076

.details__info--right {

Здесь нужно добавить отключение правого бордера

Recommend Projects