Ocena strony.

Question

Witam napisałem stronę i proszę o jej ocenę. Wiem, że nie jest jakaś super, ale zrobiłem ją w taki sposób jaki umiem.

http://grypcc.cba.pl/

Answer 1

• No i na co ta animacja wejścia? Tylko atak epilepsji u kogoś wywoła…
• Nie masz HTTP/2, więc arkusze stylów wypada złączyć w jeden i zminifikować.
• Czcionki z Google Fonts można w jedno żądanie przerobić (wystarczy poczytać w docsach jak to zrobić).
• Ostry divitis. Jak piszesz w HTML5, pisz w HTML5 → http://tutorials.comandeer.pl/html5-blog.html
• iditis. Nie powinno się stylować po [id] → http://forum.pasja-informatyki.pl/109776/html-class-czy-id?show=109816#a109816
• #all nie jest Ci wgl do szczęścia potrzebny. Możesz z powodzeniem wywalić.

• <div id="logo">
          
              <img src="logo.png" alt=""/>
      
      </div>

  To powinien być główny nagłówek strony → http://www.webkrytyk.pl/krytyka/lexy.com.pl/#naglowki dodatkowo podlinkowany do strony głównej. Co więcej: kluczowe elementy strony NIE MOGĄ MIEĆ PUSTEGO ATRYBUTU [alt]. W tym wypadku odpowiedni będzie tytuł witryny (nie zaś "Strona logo", jak ostatnio często). Można sobie poczytać: https://www.w3.org/TR/html5/embedded-content-0.html#alt

• Menu to nie jest jednak lista uporządkowana, a nie uporządkowana, zatem ul raczej bardziej tu pasuje.

• <div style="clear:both;"></div>

  Pusty element = zbędny element. Zwłaszcza, że są lepsze sposoby → http://www.forumweb.pl/porady-i-tutoriale-www/css-clearowanie-czyli-problemy-z-float,47914 http://nicolasgallagher.com/micro-clearfix-hack/

• A ten formularz kontaktowy to po co w kodzie?

Comments

O ile się orientuję to ten formularz kontaktowy dodaje sobie cba, nie wiem dlaczego i po co... ale dodaje ;) Na wszystkich stronach hostowanych na cba go spotkasz.

---

WTF? O.o Kiedyś to był w miarę porządny hosting, teraz widzę, że się syf z tego zrobił…

---

Dziękuje za ocenę.

Answer 2

Zamiast:

<div style="clear:both;"></div>

używaj:

overflow: hidden;

Np.:

http://codepen.io/anon/pen/wGejZe

Answer 3

A co tu oceniac? kilka divów, proste menu i gotowy skrypt animacji wczytywania..

Answer 4

Po 1. W zakładce "poznaj nas" mógłbyś dodać więcej informacji na temat kaczora i świstaka :P.

Po 2.W zakładce premiery 2016 to dodaj konkretną date a jeżeli jest ona nie znana to napisz "data nie znana"

A reszta myśle ok :)

 

Pozdrawiam! :D

Answer 5

Ten pattern tak średnio mi pasuje do stylistyki strony a co dopiero do tematyki. ;/

Brak RWD/AWD.

Osobny arkusz styli dla animacji (po co?)

Firebug twierdzi, że index.html jest skryptem? (Błąd?).

Answer 6

Wygląda na niedokończoną... Co tu robią te "Lorem ipsum"? No i mógłbyś zmienić styl bo to koszmarnie wygląda.