Skip to content

Добавляет микросетку главной и каталога#1

Open
elvishakirova wants to merge 32 commits into
htmlacademy-htmlcss:masterfrom
elvishakirova:master
Open

Добавляет микросетку главной и каталога#1
elvishakirova wants to merge 32 commits into
htmlacademy-htmlcss:masterfrom
elvishakirova:master

Conversation

@elvishakirova
Copy link
Copy Markdown

No description provided.

Comment thread index.html Outdated
<nav class="navigation">
<ul class="navigation-list">
<li class="navigation-item">
<a class="navigation-link" href="#">
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 index.html Outdated
</li>
<li class="navigation-item"><a class="navigation-link" href="#">Главная<a></li>
<li class="navigation-item"><a class="navigation-link" href="#">О Седоне</a></li>
<li class="navigation-item"><a class="navigation-link" href="hotels.html">Гостиницы</a></li>
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Поменять файл hotels.html на catalog.html

Comment thread index.html Outdated
<li class="navigation-item"><a class="navigation-link" href="hotels.html">Гостиницы</a></li>
</ul>
<ul class="navigation-list navigation-user">
<li class="navigation-item"><a class="navigation-link" href="#">
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 index.html Outdated
</header>

<main class="main-index">
<h1 class="visually-hidden">Добро пожаловать в Седону</h1>
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Обернуть в section

Comment thread index.html Outdated

<main class="main-index">
<h1 class="visually-hidden">Добро пожаловать в Седону</h1>
<img class="greetings" src="#" alt="Добро пожаловать">
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 index.html Outdated

<section class="about-sedona">
<h2 class="visually-hidden">О Седоне</h2>
<h3 class="about-sedona-title">Седона — небольшой городок в Аризоне, заслуживающий большего!</h3>
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Поменять на тег <p>

Comment thread index.html Outdated
<h2 class="visually-hidden">Преимущества Седоны</h2>
<ul class="advantages-list">
<li class="advantages-item">
<h4 class="advantages-item-title">Настоящий городок</h4>
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

заголовок <h3>

Comment thread index.html Outdated
<ul class="recomendations-list">
<li class="recomendations-item">
<img class="recomendations-item-image" src="#" alt="Домик">
<h4 class="recomendations-item-title">Жильё</h4>
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Поменять на <h3>

@elvishakirova elvishakirova changed the title Добавляет имя студента, наставника и название проекта Создает страницу каталога May 2, 2026
@elvishakirova elvishakirova changed the title Создает страницу каталога Добавляет изображения на главную и в каталог May 5, 2026
Comment thread images/favicon.ico Outdated
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

favicon.ico файл всегда размещается в корне проекта, фавиконки другого формата можно хранить в любой папке

Comment thread images/house-icon.svg Outdated
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 images/youtube-icon.svg Outdated
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 Outdated
<legend class="catalog-filter-title">Инфраструктуры:</legend>
<ul class="catalog-filter-list">
<li class="catalog-filter-item">
<input class="control-input" type="checkbox" name="pool" id="pool" value="pool" checked>
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Если у группы чекбоксов разное значение атрибута name, то атрибут value задавать не нужно

Comment thread catalog.html Outdated
<option value="expensive">Сначала дорогие</option>
</select>
<a class="sorting-button" href="catalog.html?view=tile">
<span class="visually hidden">Показать карточками</span>
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

класс visually-hidden является единым, для разделения слов нужно использовать дефис

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 Outdated
</label>
</li>
<li class="catalog-filter-item">
<input class="control-input" type="range" name="price" min="0" max="12000" value="0" step="1000">
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 Outdated
</li>
<li class="catalog-filter-item">
<label class="control">
<input class="control-input" type="number" name="maximum-price" placeholder="9000">
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, step атрибуты

@elvishakirova elvishakirova changed the title Добавляет изображения на главную и в каталог Добавляет стили в CSS May 12, 2026
Comment thread styles/styles.css Outdated
display: swap;
}

@font-face {
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Убрать лишние подключения italic

Comment thread styles/styles.css Outdated
font-style: italic;
font-weight: 700;
src: url("../fonts/ptsans-700.woff2") format("woff2");
display: swap;
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 styles/styles.css
text-align: center;
border: none
}

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 styles/styles.css
color: #333333;
margin: 0;
padding: 0;
text-align: center;
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Убрать выравнивание по центру для body

Comment thread styles/styles.css Outdated
font-weight: 700;
color: #000;
text-align: center;
color: #fff;
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Вместо 3 чисел указывать 6 для единообразия

@elvishakirova elvishakirova changed the title Добавляет стили в CSS Добавляет сетку главной и каталога May 18, 2026
@Sanwed
Copy link
Copy Markdown

Sanwed commented May 21, 2026

  • Вынести логотип из списка в хедере
  • Добавить отступы на ссылки в навигации, вместо тегов li
  • В элементах advantages-list указать min-height вместо height

Задать в начале css файла:
* { box-sizing: border-box }

@elvishakirova elvishakirova changed the title Добавляет сетку главной и каталога Добавляет микросетку главной и каталога May 24, 2026
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