Skip to content

первый пулл реквест#1

Open
Kondratie wants to merge 4 commits into
htmlacademy-htmlcss:masterfrom
Kondratie:project-work
Open

первый пулл реквест#1
Kondratie wants to merge 4 commits into
htmlacademy-htmlcss:masterfrom
Kondratie:project-work

Conversation

@Kondratie

Copy link
Copy Markdown

No description provided.

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

У всех иконок должно быть соответствуещее им название

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Проверить все иконки

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Опечатка
Переименовать в footer-back

Comment thread images/index/header w.svg

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Имена задаем без пробелов, слова резделяем дефисом

Comment thread catalog.html
<nav class="navigation">
<ul class="navigation-list">
<li class="navigation-item">
<a class="navigation-link" href="#">Главная</a>

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Задать корреткные ссылки в атрибуте href всех элементов навигации

Comment thread catalog.html
<a class="catalog-navigation-link" href="#">Гостиницы</a>
</li>
</ul>
<form class="catalog-filter" action="#" method="get">

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Лучшие задать форме action, который указан в задании? чтобы можно было проверить, какие данные уходят на сервер

Comment thread catalog.html
<fieldset class="catalog-filter-price">
<legend class="catalog-filter-title">Стоимость, Р:</legend>
<div class="catalog-inputs">
<input class="catalog-min" type="text" name="min-sum" value="0">

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Для полей ввода цены лучше использовать input type="number"

Comment thread catalog.html
<span class="found-hotel">38</span>
</p>
<select class="select-control">
<option value="">Сначала дешевые</option>

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

value в теге option не должно быть пустым

Comment thread catalog.html
</div>
<ul class="product-list">
<li class="product-item">
<img class="product-img" src="images/catalog/Amara-Resort-Spa.jpg" width="300" height="212" alt="гостиница Amara Resort & Spa">

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

В карточке гостиницы ссылкой для перехода на саму гостиницу должны также быть изображение и заголовок

Comment thread catalog.html
<a class="pagination-link" href="#">4</a>
</li>
<li class="pagination-item">
<a class="pagination-link" href="#">. . .</a>

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Убираем href у ссылки, т.к. троеточие является декоративным элементом

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants