ihabia,
1 - кто сказал что логотип должен быть h2? вот есть чеклист верстки https://github.com/ihorzenich/html5checklist/ это не значит, что я на 100% согласен, нет, но там есть пример, лого на главной h1, дальше лого - див. h1 - на остальных, название страницы. Такое, по крайней мере раньше, можно было везде увидеть на забугорных сайтах.
2 - вы делаете стилистику заголовков, но кроме h1, h2 и тд вы делаете .h1, .h2 - классы, и если вам надо поставить стилистику заголовка, ставите класс. Потому что расположение заголовков должен определять сеошник после анализа сайта.
- т.е. header вы назвали header-section, а вложенный контейнер .header? Где логика?
- вы написали, Логотип - это самостоятельный блок, так почему у него 2 класса, а не один logo?
- ну и лого h2, это уже вопрос к выше стоящему. Как может быть h2 выше h1? Вам сеошник по рукам надает.
вы логику напишите? что значит контент? обычно делают прилипающий блок при прокрутке, но зачем 100% высоты? или это родитель, а прокручивается внутренний блок?
ну тут скорее всего имеется ввиду как раз и вынос отдельных блоков, например, шапки, подвала, т.е. общих в отдельные файлы, что бы удобнее было редактировать в одном месте, а не на каждой странице.
прогоните svg через сервис минификатор, https://jakearchibald.github.io/svgomg/ он удаляет лишнее, и попробуйте подключить его.
Например у меня было так, что свг не работал в Firefox, прогнали, заработало.