на фоне фотку убрать, она там не нужна, у тебя должен быть акцент на сами работы, подчёркивание в заголовке убрать, убрать с фона в открытой работе фото. Мелкие чёрточки в меню сбивают с толку, зачем они там?
В футере подчёкивание “десигн бай…” - убрать, это не ссылка и не активный элемент, то же самое с левой частью в футере.
Градиенты на некоторых работах непонятно зачем, что они там делают, что они выделяют? Если ничего - убрать.
Правило близости нарушено. У тебя заголовок ближе к хедеру, чем к контенту, должно быть наоборот.