ratatujef

Нормальность обратно пропорциональна перфекционизму в этом примере. То, что у Вас элемент вышел за границы блока может добавить головной боли Вам или тому кто будет дорабатывает после Вас верстку. Я бы избегал.

Pop-up окно это такой же обычный блок html как и все остальный.
Его базовые стили:

position:absolute;
display: none;

Поэтому в него вы можете спокойно вставить html-form в pop-up c кнопкой[type='submit'] и все будет работать.
При активации будет меняться display: block, и pop-up будет появляться с формой внутри.

Разумеется полностью за вас никто делать задание не будет. Но наводящий пример сделать можно - codepen. Общая идея должна быть понятна: с помощью v-model связываем данные с инпутами, с помощью computed-свойств проверяем данные на правильность. В примере только одно поле, вам нужно его расширить на несколько.

Привет.
В первую очередь лучше просить обратную связь у компании.

Что меня смущает:
– почему не смиксовать классы header__wrapper и container (то же самое в футере)
– какой-то странный нейминг menu__item__link, news__item__subheader и другие: двумя нижними подчеркиваниями в данном примере разделяются элемент и блок, скорее всего подразумевалось menu__item-link (в других вариантах так же)
– title_main и title_news очень разные сущности, они не подходят под модификатор блока title
– "Новости" точно заголовок. Максимум p или span, но никак не div
– аналогично везде, где текст: div не имеет семантику, а просто блочная обертка. Для текста используются соответствующие теги: h1-h2, p, i, b и т.д.
– карточку можно сделал отдельным блоком, а так же использовать тег article

К чему еще можно придраться
– "Лидер в разработке..." мне кажется больше на параграф тянет, чем на h1
– возможно ожидали, что картинка под "Лидер в разработке..." должна прописываться в html
– могли не понравиться орфографические ошибки (например, комментарии)
– лучше использовать текст из макета