Prośba o ocenę strony

[SebastianPob]

Ucze sie od jakichś 7 miesięcy.

http://codepen.io/SebastianPob/pen/LkwAmZ

Konstruktywna krytyka mile widziana

 

[Burned123]

Jak zobaczyłem nawigacje zbudowaną z div > h3 to się załamałem. Tyle czasu nauki poszło w pizdu :/

[vPaN]

Ocenię tak na szybko, nie wchodząc "głębiej" (powierzchownie przelece, błęy najbardziej rzucające się w oczy)

 

 

1. Po pobraniu paczki z codepen na pulpit mam dziwny template --- doctype -> html -> head -> body -> head -> body itd, hm? ;d
2. Fajnie, że określiłeś język tylko dlaczego jest "pl" na stronie po angielsku?
3. Charset pierwszy w head, za to +
4. Chrome frame nie istnieje od lat
5. Preloader to tylko niepotrzebny dodatek + pusty div to zbędny div + stylowanie inline
6. Skrypty porozrzucane po całym dokumencie, ciężko cokolwiek powiedzieć przy takiej strukturze
7. Do <script> nie ma potrzeby dodawania type="js", większość przeglądarek nie potrzebuje tej informacji (automat)
8. Nieprawidłowa hierarchia nagłówków + kolejne stylowanie inline
9. Formularz jest oczywiście niedostępny (brak label)
10. Zamiast onclick użyj addEventListener
11. Kolejne puste div'y
12. Zadeklarowany html5 a nie korzystasz z jego dobrodziejstw, same div'y

Jest tego więcej, ale nie mam czasu, żeby o tym pisać.

Jak na 7 miesięcy nauki to kod jest tragiczny, polecam poczytać o html5, semantyce oraz zasadach choćby podstawowej dostępności dla osób korzystających z czytników ekranowych lub osób korzystających z klawiatury, nie sprawdzałem czy działa nawigacja tab'em i czy jest focus, ale ktoś może to sprawdzić i napisać.

Miłego wieczoru

[QuarK]

Zamykam na prośbę autora (raport).