Здравствуйте!
Пока тоже учусь. Из того, что заметила:
1. При верстке блоков с текстом учитывайте, что его размеры могут меняться. Чтобы так не было, например:
2. Меня смущает горизонтальный скролл, который вылезает на доли секунды при прокрутке.
3. Тут ожидается, что в аккордеоне справа при нажатии на стрелочку пункт будет схлопываться, но этого не происходит
4. Здесь в карусели невозможно курсором выделить текст, чтобы скопировать. Я бы не стала блокировать эту возможность
5. Насколько я понимаю, картинки, содержащие информацию, лучше верстать через тег img, а не свойство background-image. Здесь у Вас фото как фон (кстати, там же и проблема с размером подписи)
6. У меня нет макета, но есть подозрение, что стрелочка в круге должна быть по центру, а не смещена вниз
7. В галерее "some of our work" меня смущают три вещи: 1) фото невозможно увеличить, хотя при этом на сайте много всякой анимации; 2) столбцы имеют класс row, тогда как обычно он используется для названия строк (для столбцов обычно col); 3) картинки снова сверстаны как фоны.
8. Я не вполне согласна с использованием тегов в блоге (основной блок и в футере) с точки зрения семантики. Весь блог я бы обернула в section, а превьюшки сделала через article каждую. И по меньшей мере с фотки должна быть ссылка на статью.
9. При наведении на анимированные элементы курсор ведет себя по-разному, хотя во всех этих случаях при клике ничего не происходит. Например,
10. При прокрутке вниз, на мой взгляд, слишком большая задержка у появления элементов. Я не знаю, какая должна быть, но вот как пользователя напрягает
В остальных вопросах не могу оценить. Пока не умею, как Вы. Но есть вопрос: кроме .container, других классов нет, определяющих сетку? Я правильно поняла, что все построено на flexbox?