Skocz do zawartości

qlzone.xorg.pl - Quake LIVE Zone


ShadoQ

Recommended Posts

Stronka wizualnie mi się podoba. Niestety jeśli chodzi o validację to nie jest tak kolorowo: 177 Errors, 99 warning(s)! Popraw kodowanie. Poza tym kiedy oglądam źródło strony to:

*Gdzie deklaracja typu dokumentu oraz "</html>" ?

*</BODY></CENTER></CENTER></td> - body powinno być tylko jedno!

*<BODY bgcolor="#000000" - Nie dość, że jest to drugie body to jeszcze nie domknięte.

*<?php echo </CENTER> - Co to jest?

*Menu na obrazkach? Niedobry pomysł. Żadna wyszukiwarka tego nie odczyta.

Oprócz tego wszystkiego strona jest zbudowana na tabelkach a to nie jest najlepsze rozwiązanie. Wydłuża czas ładowania strony oraz staje się one mniej widoczna dla wyszukiwarek. Jeśli chodzi o treść to nie ma jej za wiele, ale jeśli strona jest nowa to można to wybaczyć.

Moja ocena: 4/10 i te 4 jest za wygląd oraz treści pojawiające się na stronie głównej.

  • Upvote 1
Link do komentarza
Udostępnij na innych stronach

Witaj na forum!

Widać, że strona jest nowa, ma niewiele ponad miesiąc. Strona wizualnie wygląda w porządku, jednak w kodzie jest sporo błędów.

Pierwsze co rzuca się w oczy to już druga linijka gdzie lang="fr". Parametr lang definiuje język w jakim strona jest napisana więc dla polskiej strony jest to oczywiście "pl" i powinien być on wstawiony w tagu html:

&lt;html lang="pl"&gt;

Idąc parę linijek dalej można zauważyć, że omijasz "#" (krzyżyk) przy wpisywaniu kolorów, oraz raz raz podajesz parametry używając raz cudzysłów, raz apostrof.

W kodzie jest pełno błędów które powinieneś poprawić myśląc poważnie o stronie. Popraw te błędy napisane przez WisJa oraz mnie i odezwij się, to chętnie wskażę kolejne:). Jeżeli będziesz miał problemy, napisz w odpowiednim dziale forum - chętnie pomożemy!

  • Upvote 1
Link do komentarza
Udostępnij na innych stronach

Widzę, że 20 errorów zniknęło:) Jeszcze parę rzeczy, które widzę:

*Zamiast <html> daj:

&lt;!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd"&gt;
&lt;html xmlns="http://www.w3.org/1999/xhtml"&gt;

*Co to jest topmargin? Zamiast tego:

&lt;body id="v-xmas-2009" text="000000" link="1B5472" topmargin="0"&gt;

daj w CSS:

body{
  margin-top: 0;
  color: #000;
}
a, a:visited
{
  color: #1B5472;
}

*Co to jest: </gif> ?

*<br> jest znacznikiem pustym, tzn zamiast <br> powinni być odrazu zamknięcie czyli <br/>

I jeszcze dużo drobnych błędów. Ale postępy są :) Oby tak dalej :D

Link do komentarza
Udostępnij na innych stronach

×
×
  • Utwórz nowe...