Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Микросетки на флексах для всех страниц #6

Open
wants to merge 25 commits into
base: master
Choose a base branch
from

Conversation

Alexandra0107
Copy link
Contributor

@Alexandra0107 Alexandra0107 commented Nov 12, 2024

…", разметка отображения карточек+нумерация страниц
@keksobot keksobot changed the title Корректировка названия картинка ТГ, корректировка тег form в "Каталог… Графика для всех страниц Nov 12, 2024
@Alexandra0107
Copy link
Contributor Author

Оптимизация картинок

Catalog.html Outdated
<div class="infrastructure">
<input type="checkbox" name="infrastructure" value="pool" id="infrastructure-pool" checked>
<input type="checkbox" name="infrastructure-pool" value="pool" id="infrastructure-pool" checked>

Choose a reason for hiding this comment

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

В checkbox используй один name, при выборе значений, будет происходит группировка по имени, то есть конечный результат name = [ список значений ]

Catalog.html Outdated
</main>
<footer class="main-footer">
<ul class="social-list">
<li class="social-list-item">
<a href="https://vk.com/htmlacademy"><img src="./img/social_vk.svg" width="24.0001" height="14" alt="VK"></a>
</li>
<li class="social-list-item">
<a href="https://t.me/htmlacademy"><img src="./img/social _tg.svg" width="22" height="17" alt="Telegram"></a>
<a href="https://t.me/htmlacademy"><img src="./img/social_tg.svg" width="22" height="17" alt="Telegram"></a>

Choose a reason for hiding this comment

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

Названия картинок через дефис

index.html Outdated
Comment on lines 42 to 47
<div class="welcome">
<div class="welcome2">
<div class="welcome3">
</div>
</div>
</div>

Choose a reason for hiding this comment

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

Чутка перебор с вложеностью, тут мы будет использовать img, так что нам будет достаточно 1 секции и нужного тега внутри

@keksobot keksobot changed the title Графика для всех страниц Базовая стилизация всех страниц Nov 19, 2024
keksobot pushed a commit that referenced this pull request Nov 19, 2024
@keksobot
Copy link
Contributor

♻️ Я собрал ваш пулреквест. Посмотреть можно здесь.

Catalog.html Outdated
@@ -181,14 +188,36 @@ <h2>Подпишитесь на рассылку</h2>
<input type="submit" value="Подписаться">
</form>
</section>
<section>

Choose a reason for hiding this comment

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

Это больше будет nav, так как тут у нас навигация

css/style.css Outdated
text-decoration: none;
}

h1 {

Choose a reason for hiding this comment

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

Избегай селекторы по тегам, академия это бракует, лучше заведи общий класс


}
.features-list-item{
padding: 80px 85px 80px 85px;

Choose a reason for hiding this comment

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

Стиль кода

index.html Outdated
<p class="reasons"> Опытный персонал и качественное обслуживание!</p>
<ul class="features-list">
<li class="features-list-item">
<img src="./img/advantage-icon-1.svg" alt="Жилье">

Choose a reason for hiding this comment

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

Это будут иконки, то есть декоративные изображения, а не img

index.html Outdated
</li>
<li class="social-list-item">
<a href="https://www.youtube.com/user/htmlacademyru"><img src="./img/youtube.svg" width="22" height="17"
<a contenteditable="youtube" href="https://www.youtube.com/user/htmlacademyru"><img src="./img/youtube.svg" width="22" height="17"

Choose a reason for hiding this comment

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

Так же будут иконки

@keksobot keksobot changed the title Базовая стилизация всех страниц Сетка страниц и крупных блоков на флексах Nov 28, 2024
@keksobot
Copy link
Contributor

keksobot commented Dec 2, 2024

♻️ Я собрал ваш пулреквест. Посмотреть можно здесь.

keksobot pushed a commit that referenced this pull request Dec 2, 2024
}

.page-footer-container {
width: 1200px;

Choose a reason for hiding this comment

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

Ты можешь поставить ограничения на самом верхнем уровне, чтобы не прописывать каждому элементу + выравнивание по центру

background-color: #fff;
display: flex;
flex-direction: column;
min-height: 100%;

Choose a reason for hiding this comment

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

Используй лучше 100dvh вместо %

}

.blok-1 {
color: #FFFFFF;

Choose a reason for hiding this comment

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

В обном стиле, у тебя где-то верхний регистр, а где-то нижний

css/style.css Outdated
display: flex;
flex-wrap: wrap;
background-color: #82B3D3;
height: 384px;

Choose a reason for hiding this comment

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

Не здавай фиксированные значения высоты контентным блокам, они не пройдут проверку на переполнение, отдавай приоритет min-height, только декоративным элементам фиксируй height

css/style.css Outdated
text-transform: uppercase;
}

a {

Choose a reason for hiding this comment

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

Старайся избегать теги, академия это не любит, лучше класс + все общие стили на верх перенеси

Choose a reason for hiding this comment

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

*-icon-3, проверь именования

index.html Outdated
Comment on lines 47 to 53
<div class="page-container">
<div class="welcome">
<div class="welcome2">
<div class="welcome3">
</div>
</div>
</div>

Choose a reason for hiding this comment

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

Зачем такая вложенность?

index.html Outdated
</div>
<ul class="features-list">
<li class="features-list-item">
<img src="./img/advantage-icon-1.svg" alt="Жилье">

Choose a reason for hiding this comment

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

Сейчас это не критично, но к концу, нужно переделать на иконку, и убрать тег img, академия придерется

<svg width="23" height="18" viewBox="0 0 23 18" fill="none" xmlns="http://www.w3.org/2000/svg">
<path
d="M18.9402 0.356445H3.50738C1.64668 0.356445 0.333252 1.9502 0.333252 3.75645V13.8502C0.333252 15.7627 1.64668 17.3564 3.50738 17.3564H19.1591C20.8009 17.3564 22.3333 15.7627 22.3333 13.9564V3.75645C22.1143 1.9502 20.8009 0.356445 18.9402 0.356445ZM7.99494 12.8939V4.81894L15.3283 8.85645L7.99494 12.8939Z"
fill="#83B3D3" />

Choose a reason for hiding this comment

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

Во всех svg используй для цвета fill currentColor, это нам прогодится, при изменении состояний

@keksobot keksobot changed the title Сетка страниц и крупных блоков на флексах Микросетки на флексах для всех страниц Dec 9, 2024
@keksobot
Copy link
Contributor

♻️ Я собрал ваш пулреквест. Посмотреть можно здесь.

keksobot pushed a commit that referenced this pull request Dec 11, 2024
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.

3 participants