code review strony

pytanie zadane 9 lutego 2020 w HTML i CSS przez Stoprocent Użytkownik (930 p.)

Witam, prosze o pomoc w wyłapaniu największych byków. jedne z pierwszych kroków z Bem i scss.

ps(proszę tylko pominąć tego slidera ^^)

kod

live

2 odpowiedzi

odpowiedź 9 lutego 2020 przez Shaoi Mądrala (7,020 p.)

<!--[if lt IE 9]>
	<script src="https://cdn.jsdelivr.net/npm/html5shiv@3.7.3/dist/html5shiv-printshiv.min.js"></script>
<![endif]-->

Brakuje wsparcia dla IE (nie ma tego u Ciebie);

```
<a href="#main" class="focus-only"></a>
```

Tego również brakuje;

Brakuje id dla części tagów;
Brak aria-controls dla buttona;
nav wstawiłbym do header;
Warto dodać ukryty nagłówek dla nav;
div.navigation__logo zamieniłbym na h1;
header.hero zamieniłbym na section;

<span class="hamburger__inner"></span>

<i class="fa fa-long-arrow-right"></i>

Raz robisz to dobrze, a innym razem już źle. https://blog.comandeer.pl/o-ikonkach-slow-kilka.html;

div.*__heading zamieniłbym na coś bliżej tego:

<header class="section__header">
	<h2></h2>
	<p></p>
</header>

<div class="overlay" id="overlay"></div>

NIE NIE NIE NIE!

komentarz 9 lutego 2020 przez Tomek Sochacki Ekspert (227,510 p.)

Brakuje wsparcia dla IE (nie ma tego u Ciebie)

nie pisz, że brakuje skoro nie znasz założeń biznesowych aplikacji :) napisz po prostu, że strona może nie mieć wsparcia dla IE jeśli nie dasz polyfill itp. ale sam brak takiego wsparcia niekoniecznie jest czymś złym, sam coraz częściej spotykam się z decyzjami biznesowymi, że wiele nowych ficzerów możemy powoli odcinać od IE11, a starszych w ogóle nie wspierać. Oczywiście zawsze jest dylemat biznesowy czy starata tych X userów jest do zaakceptowania, ale jak pisałem, sam fakt braku wsparcia to nie błąd jeśli nie znamy decyzji biznesowych w tych tematach.

<div class="overlay" id="overlay"></div>

NIE NIE NIE NIE!

jak piszesz, że NIE to napisz co konkretnie Ci nie pasuje :)? Nie patrzę teraz w kod bo jestem na mobilce, więc nie wiem czego się to tyczy w kodzie apki ale sama składnia tego div jest jak najbardziej poprawna.