Niepoprawny.
Po pierwsze, jest to przycisk. Dlaczego zatem nie oznaczyłeś go przy pomocy elementu button?
Po drugie, przycisk ten nie powinien być zamieszczony jako pierwszy element w liście zawierającej linki menu, moim zdaniem z punktu widzenia struktury treści to nie ma sensu. Ja bym proponowała coś takiego:
<nav class="header-nav nav">
<button class="menu-toggle">Menu</button>
<ul class="header-nav-ul ul">
<li class="menu"><a href="#">Home</a></li>
<li class="menu"><a href="#">Blog</a></li>
<li class="menu"><a href="#">Portfolio</a></li>
<li class="menu"><a href="#">O Mnie</a></li>
<li class="menu"><a href="#">Kontakt</a></li>
</ul>
</nav>
Nie zapomnij też dodać odpowiednie atrybuty ARIA (aria-expanded, aria-controls, ewentualnie aria-pressed).
I jeszcze przy okazji:
<h1 class="header-logo logo"><img src="" alt=""></h1>
Z punktu widzenia osób korzystających z czytników ekranu nagłówek ten jest pusty; nie posiada ani tekstu, ani alternatywnego opisu dla logo.