Code review #2

[ZvezdocheTag]

1. У тебя в данный момент html фаил называеться Holydays.html, инициализационный файл заглавной html страницы всегда должен называться index.html,

2. Я проверил адаптив в эмуляторе браузера хром, он там не работает потому что в html > head не указан мета тег для поддержки адаптива, <meta name="viewport" content="width=device-width, initial-scale=1.0"> он подсказует браузеру на каком устройстве открыт твой сайт и в соответствии с этим применяет адаптив

3. MyFirstRep/HTML Module 4 HomeWork от 16,09/Holydays.html

   Line 15 in e265a4f

   <div class="contacts">

   
   В данном блоке не осмысленные названия детей contacts ,задавать названия разметки нужно логично на это обращают первое внимание, можно как то так:

					<div class="contact__title">Наши Контакты:</div>
					<div class="contact__phone contact__phone--red">+9-098-765-4321</div>
					<div class="contact__phone">+9-0123-456-890</div>
					<div class="contact__mail">
						<a href="#">info@asia-holidays.com</a>
					</div>

4. MyFirstRep/HTML Module 4 HomeWork от 16,09/Holydays.html

   Line 23 in e265a4f

   <div class="countries">

   
   тут тоже вопрос с именованием, смотри логику у тебя есть список стран которые нужно отобразить каждый элемент списка будет иметь одну и ту же структуру, соответственно чтобы не дублировать код тебе можно всем этим элементам задать один и тот же класс. то есть не countries__india,iran ... а countries__item, а если отдельно нужно выделить какую то странну то тогда задаешь ей дополнительный класс с модификатором типа countries__item countries__item--active

5. MyFirstRep/HTML Module 4 HomeWork от 16,09/Holydays.html

   Line 13 in e265a4f

   <img class="logo" src="img/logo.png" alt="">

   
   Лого должно быть обернуто в ссылку, так как зачастую если у тебя несколько страниц оно ведет на главную страницу

[RusRogo]

Все исправил, по БЭМ-у переделал, спасибо за науку.
Модификатор класса типа "countries__item--active" - Sublime Text не отображает подсказки-ссылки с HTML на CSS. Вообще со знаком "-" в классе не хочет работать.
Надо на Visual Studio Code переходить?

[RusRogo]

Папка - "homework_16.09--corrected_20.09"

[ZvezdocheTag]

Visual Studio набирает оборотов и с ней удобно работать, пока для верстки с головой достаточно sublime, но чуть позже когда начнешь работать с пакетами и сборщиками VS code в значительной мере упростит жизнь

[ZvezdocheTag]

Заметка по БЭМ ты создал один общий класс .countries__item и потом добавил с модификатором .countries__${some_country}_placement, соответственно все общие стили можно определить в .countries__item, а уникальность в уникальный класс . В данном примере заметь у тебя постоянно дублируеться свойство top: 131px

MyFirstRep/homework_16.09--corrected_20.09/css/st.css

Line 84 in 8a3ac57

.countries__item {