Сетка страниц и крупных блоков на флексах #5
Conversation
keksobot
changed the title
|
♻️ Я собрал ваш пулреквест. Посмотреть можно здесь. |
index.html
Outdated
| @@ -7,6 +7,7 @@ | |||
| </head> | |||
| <body> | |||
| <header class="page-header"> | |||
| <div class="page-header-container"> | |||
There was a problem hiding this comment.
Нужно соблюдать табуляцию и вложенность. Так код будет легче воспринимать и тебе и другим.
index.html
Outdated
| @@ -31,52 +32,58 @@ | |||
| </a> | |||
| </li> | |||
| <li class="navigation-item"> | |||
| <a class="button" href="#"> | |||
| <a class="navigation-link" href="#"> | |||
| <span class="visually-hidden"> | |||
There was a problem hiding this comment.
Лишний элемент. Нет необходимости дополнительно вкладывать текст в спан.
index.html
Outdated
| <span class="visually-hidden"> | ||
| Избранное | ||
| </span> | ||
| </a> | ||
| <span>12</span> | ||
| </li> | ||
| <li class="navigation-item"> | ||
| <a class="button" href="#">ХОЧУ СЮДА!</a> |
There was a problem hiding this comment.
Верхнего регистра лучше добиваться через стили (существует свойство text-transform) А в разметке лучше писать как обычно.
index.html
Outdated
| <div class="page-container"> | ||
| <div class="intro"> | ||
| <h1 class="visually-hidden">Добро пожаловать в город Седона!</h1> | ||
| <img src="images/welcome.svg" width="458" height="352" alt="Добро_пожаловать!"> |
There was a problem hiding this comment.
В alt можно использовать пробелы, если что
index.html
Outdated
| <img src="images/divider.svg" width="1200" height="57" alt="Декоративная_вставка"> | ||
| </div> | ||
| <h2 class="city-intro">Седона — небольшой городок в Аризоне,<br> | ||
| заслуживающий большего!</h2> | ||
| <p class="advantages-intro">Рассмотрим 5 причин, по которым Седона круче, чем Гранд-Каньон!</p> |
There was a problem hiding this comment.
Сбилась табуляция.
| @@ -14,7 +14,15 @@ src: url("../fonts/ptsans-700.woff2") format("woff2"); | |||
| font-display: swap; | |||
| } | |||
|
|
|||
| html{ | |||
There was a problem hiding this comment.
Давай пробел добавлять перед открывающей скобкой css-правила
| body{ | ||
| margin:0; |
There was a problem hiding this comment.
И перед значением тоже пробел.
styles/styles.css
Outdated
|
|
||
| .page-header-container{ | ||
| display: flex; | ||
| flex-direction:row; |
There was a problem hiding this comment.
Это значение по умолчанию. Его можно не прописывать.
Оставила фикс-высоту только для subitems. Не знаю, как вытянуть на такую же высоту, как и у остальных, нижний ряд items, все едет.
Та часть, которую успела сделать.
🎓 Сетка страниц и крупных блоков на флексах
💥 https://htmlacademy-htmlcss.github.io/1923813-sedona-33/5/