Forum PHP.pl > Hotel Malbork

Pomoc - Szukaj - Użytkownicy - Kalendarz

hotelmalbork

29.08.2008, 07:52:46

Witam, proszę o ocenę mojej strony Hotel Malbork Majewski

starach

29.08.2008, 09:13:35

Jak dla mnie strona świetnie wygląda, ale mam kilka zastrzeżeń.
Elementy menu ładują się dopiero po najechaniu na nie myszką. Ładuj w JSie obrazki od razu.
Oprócz tego walidatory w3.org zwracają masę błędów jak na Transitional.
Wygląd - Mnie pasuje - 8.5 / 10
Kod - 4.5 - Teraz się używa strict'a, masz burdel w kodzie, walidatory w3.org
W sumie - 6.5

JoShiMa

29.08.2008, 09:57:59

Rzut oka na kod i pierwsze co się rzuca w oczy to:

[HTML] pobierz, plaintext 
</head>
   <body>
      <center>
         <table width="968" border="0" cellpadding="0" cellspacing="0">
         ...
[HTML] pobierz, plaintext

[HTML] pobierz, plaintext 
<td colspan="16" valign="top" bgcolor="#ffc269">
    <img src="images/napisonas.jpg">
    <table> ...
[HTML] pobierz, plaintext

Kto w ten sposób dzis jeszcze pisze strony?

Następnie:

[HTML] pobierz, plaintext 
<br />
[HTML] pobierz, plaintext

W HTML 4.01 nie domyka się w ten sposób znaczników

[HTML] pobierz, plaintext 
<meta http-equiv="Content-Type" content="text/html; charset=windows-1250">
[HTML] pobierz, plaintext

Deklaracja dobrze, że w ogóle jest, ale czemu akurat taka? Czemu nie bardziej cywilizowana strona kodowa?

[HTML] pobierz, plaintext 
<br /><br />
[HTML] pobierz, plaintext

Odstępy między elementami layoutu robi się za pomocą stylów. Chyba o tym jeszcze nie słyszałeś.

[HTML] pobierz, plaintext 
<b><h1>HOTEL MALBORK MAJEWSKI</h1></b>
[HTML] pobierz, plaintext

[HTML] pobierz, plaintext 
<center>
<br /><br /><br />
<h2>W PRZYSZŁOŚCI</h2>
</center>
[HTML] pobierz, plaintext

Takie konstrukcje też są zbędne jak się zastosuje style.

A to to już w ogóle nie zgadnę do czego służy.

[HTML] pobierz, plaintext 
<td>
			<img src="images/spacer.gif" width="59" height="1" alt=""></td>
		<td>
			<img src="images/spacer.gif" width="17" height="1" alt=""></td>
		<td>
			<img src="images/spacer.gif" width="71" height="1" alt=""></td>
		<td>
 
			<img src="images/spacer.gif" width="202" height="1" alt=""></td>
		<td>
			<img src="images/spacer.gif" width="286" height="1" alt=""></td>
		<td>
			<img src="images/spacer.gif" width="11" height="1" alt=""></td>
[HTML] pobierz, plaintext

Podsumowując. Kod nędzny. Powiedzieć, że amatorski to i tak by było za wiele.

To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.