Feature/initial footer #22
Conversation
| @@ -0,0 +1,7 @@ | |||
| .contacts__phone-subheader { | |||
| font-family: 'YS Display', Arial, sans-serif, Arial, sans-serif; | |||
There was a problem hiding this comment.
font-family не стоит прописывать в каждом текстовом элементе: мы подключим его к классу тега body (т. о., убираем дублирование)
| @@ -0,0 +1,7 @@ | |||
| .contacts__phone-subheader { | |||
| font-family: 'YS Display', Arial, sans-serif, Arial, sans-serif; | |||
| font-style: normal; | |||
There was a problem hiding this comment.
Аналогично со свойствами: font-weight, font-style, font-size. Мы пропишем самый частый случай в теге body, а там, где будет отличаться, пропишем.
| font-size: 16px; | ||
| line-height: 24px; | ||
| text-align: left; | ||
| margin: 4px 0 4px 0; |
| color: black; | ||
| } | ||
|
|
||
| .link:hover { |
There was a problem hiding this comment.
где ховера нет? В макете - да, там еще ничего не прописано.
посмотрел реализацию не готовом варианте лендинга.
| <head> | ||
| <meta charset="UTF-8"> | ||
| <meta http-equiv="X-UA-Compatible" content="IE=edge"> | ||
| <meta name="viewport" content="width=device-width, initial-scale=1.0"> | ||
| <title>Document</title> | ||
| <link href="./pages/index.css" rel="stylesheet" type="text/css"> |
index.html
Outdated
| <footer class="footer"> | ||
| <div class="footer-container"> | ||
| <ul class="footer-container-column contacts"> | ||
| <li class="contacts__phone">8-800-700-93-29</li> |
There was a problem hiding this comment.
index.html
Outdated
| <li class="footer-container-column__item"> | ||
| <li class="social-items"> | ||
| <a href="" class="social-items__item">Журнал</a> | ||
| <img src="images/vk-icon.svg" alt="vkontakte-icon" class="social-items__icon"> |
There was a problem hiding this comment.
Может, так?
alt = "Иконка социальной сети %22ВКонтакте%22"
%22 - это кодировка кавычек
index.html
Outdated
| <li class="social-items"> | ||
| <a href="" class="social-items__item">Журнал</a> | ||
| <img src="images/vk-icon.svg" alt="vkontakte-icon" class="social-items__icon"> | ||
| <img src="images/telegram-icon.svg" alt="telegram-icon" class="social-items__icon"> |
index.html
Outdated
| </li> | ||
| <li class="copyright"> | ||
| <p class="copyright__item">АНО ДПО «ШАД», ООО «Яндекс»</p> | ||
| <p class="copyright__item">©2022</p> |
There was a problem hiding this comment.
Здесь был бы уместен семантический тег: https://developer.mozilla.org/ru/docs/Web/HTML/Element/time
vendor/normalize.css
Outdated
| /* Отступ слева в браузере Firefox, Safari, Chrome */ | ||
| } | ||
| /* Сброс подчеркивания у ссылок */ | ||
| a { |
There was a problem hiding this comment.
Не задумывался над тем, что это можно прописывать в нормалайзере
There was a problem hiding this comment.
я не знаю))
подумал, что это удобно, поэтому прописал
No description provided.