# TASL Review ``user-scalable=no`` - usunąłbym to z ``<meta name="viewport" />`` z punktu widzenia dostępności zabranianie użytkownikowi zoom’owania strony nie jest dobre :) ```html <header> <nav> <img src="assets/img/logo_png/logo.png" alt="TASL Logo" class="logo_png" /> <div id="nav-icon3"> <span></span> <span></span> <span></span> <span></span> </div> </nav> </header> ``` Biorąc pod uwagę semantykę i dostępność przycisk Hamburger powinien być buttonem bo takie też ma zadanie :) I nie powinno to być w ``<nav>`` jak dla mnie :P Klasa **content_container** myślę że śmiało może nazywać się **container** ```html <div class="slider_container"> <div class="center slider"> <div> <h1>1</h1> </div> <div> <h1>2</h1> </div> <div> <h1>3</h1> </div> <div> <h1>4</h1> </div> <div> <h1>5</h1> </div> <div> <h1>6</h1> </div> <div> <h1>7</h1> </div> <div> <h1>8</h1> </div> <div> <h1>9</h1> </div> </div> </div> ``` Nie wiem czy to tylko placeholder, w każdym razie **h1** powinien występować na stronie tylko raz :) ```html <div> <a href="">Employers</a> <a href="">Schools</a> <a href="">Rewards Providers</a> </div> <div> <a href="">How It Works</a> <a href="">About</a> <a href="">Contact</a> <a href="">Subscribe</a> </div> <div> <a href="">FAQ</a> <a href="">Terms & Conditions</a> <a href="">Privacy</a> </div> ``` To powinno być owinięte w ``<nav>``, a także zamiast div’ów powinieneś użyć listy ``<ul>`` :) ```js $('#nav-icon1,#nav-icon2,#nav-icon3,#nav-icon4').click(function(){ $(this).toggleClass('open'); }); ``` Na stronie używasz tylko ``#nav-icon3`` :) Poza tym skrypt warto by przenieść do index.js, aby zachować _separation of concerns_. Od siebie proponował bym jeszcze przyjrzeć się BEM [Quick start / Methodology / BEM](https://en.bem.info/methodology/quick-start/)