# 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/)