Layout nie przypadł mi do gustu. Po poprawkach byłby IMHO ok (za duży logotyp, nieujednolicone menu, położenie wyszukiwarki, mało widoczny przycisk RSS, zbyt powolny fadeOut rozwijanego menu, brak wyróżnienia obecnej podstrony w menu).
Jeżeli chodzi o kod:
- brak meta o RSS/ATOM,
- zbyt wiele plików js (pamiętaj, że zwiększy to obciążenie serwera; przeglądarki posiadają limity pobieranych zapytań),
- CSS w head,
- zalecam używanie anglojęzycznego nazewnictwa; jest to wyraz profesjonalizmu i dobre przyzwyczajenie na przyszłość jeżeli swoją karierę wiąże się wyżej),
- linki z samym obrazkiem są dla robotów prawie bezużyteczne, dlatego lepiej jest logotyp ustawić jako background,
- jeszcze bardziej bezużyteczne są puste linki,
- h1 jest tylko jeden w dokumencie (najczęściej zawiera on nazwę strony),
- staraj unikać się pustych div-ów (domyślam się po nazwie klasy div-u ?rozdzielacz?, że jego miejsce mógłby zająć hr),
- staraj się używać więcej trików w CSS-ie, aby ograniczyć niepotrzebne znaczniki (dla przykładu użycie br-ów w liście komentarzy),
-
<p style="display: none;"><input type="hidden" name="poll_id" value="2" /></p>
,
-
<p style="text-align: center;"><strong>Który element tej strony jest Twoim zdaniem najgorzej wykonany i wymaga poprawy?
</strong></p>
Nie mnóż niepotrzebnych znaczników. Unikaj dodawanie styli w HTML.
- link z kontaktowym mail-em daj jako mailto:,
- nie rozumiem dlaczego opis input-a tekstowego znajduje się po jego prawej stronie? Przyjęła się raczej lewa strona,
- wszelkie dodatki do pola textarea (emoty, bbcode) radziłby umiejscowić nad polem, a nie pod całym formularzem - jest to nieintuicyjne,
- nawigacja po podstronach jest utrudniona w przypadku braku/wyłączonej obsługi JS-a,
- jeżeli podajesz w CSS-ie np. border-radius dla gecko i webkit-a to dodaj także jako zwykły parametr - nie będziesz musiał w przyszłości zmieniać kodu i dasz szansę cieszyć się ładniejszym wyglądem użytkownikom innych przeglądarek (np. opery),
- nie mieszaj tylu czcionek naraz (Arial, Verdana, Georgia); dobrze jest ustalić czcionki dla całej strony i uwzględnić w liście czcionki alternatywne.
PS Jeżeli już ktoś trafił na twoją stronę i na prawdę interesują go artykuły to nie musisz zwracać się do niego jak do totalnego laika - on z pewnością nie poradzi sobie z kodem. Zdecyduj jaki jest twój target. Teraz piszesz jak dla laików, a serwujesz zagadnienia dla bardziej zaawansowanych.
PS2 Staraj się odchudzać kod, usuwać niepotrzebne znaczniki i zastępować div-y lepiej przeznaczonymi do danej sytuacji odpowiednikami.