<header> has divity. Remove this <div class="nav">.
GET RID OF INLINE STYLING!
<img class="header__logo" src="/assets/logo_white_small.png" alt="small logo"> has issue with BEM convention. Element shouldn't be named as HTML element. What if we changed <header> for other tag? We had rename two elements instead of one.
For code consistency - a & img should be wrapped multiline.
https://github.com/hs3city/hs3city.github.io/blob/ad43120d6dde71371a6c36af10101426db593a19/_includes/header.html#L1
<header>has divity. Remove this<div class="nav">.<img class="header__logo" src="/assets/logo_white_small.png" alt="small logo">has issue with BEM convention. Element shouldn't be named as HTML element. What if we changed<header>for other tag? We had rename two elements instead of one.a&imgshould be wrapped multiline.