[gra] Ocena wyglądu Opcje

[Neon12345]

www.handball-manager.pl- Proszę o ocenę szablonu zewnętrznego jak i wewnętrznego ( Gra )

Skala od 0/10

Ten post edytował Neon12345 9.07.2010, 08:28:19

[erix]

Z racji, że nie chce mi się rejestrować, zajrzałem pod bebechy:

[HTML] pobierz, plaintext 
<div class="FeQ">
<div class="logo">
<div class="logo-obr1"></div>
<div class="logo-obr2"></div>
<div class="logo2">
<div class="logo-obr3"></div>
<div class="button-rejestracja"><a class="no-border" href="register"><img src="img/_06.gif" alt="obrazek"></a></div>
<div class="logo-obr4"></div>
</div>
<div class="logo-obr5"><div class="login">
<div class="login-obr1"></div>
<div class="login2">
<form  action="index.php?login=yes" method="post">
<div class="login-obr2"></div>
<div class="login-obr3"><input name="user" type="text" style=" background-image: url(images_zew/_07.gif); background-repeat: no-repeat; width: 120px; height: 19px; float: left;"></div>
<div class="login-obr4"></div>
<div class="login-obr5"></div>
<div class="login-obr6"></div>
<div class="login-obr7"><input name="haslo" type="password" style=" background-image: url(images_zew/_15.gif); background-repeat: no-repeat; width: 120px; height: 19px; float: left; "></div>
<div class="login-obr8"></div>
</div>
<div class="login3">
<input type="image" src="images_zew/_09.gif"  />
</div>
<div class="login-obr10"></div>
<div class="login-obr11"></div>
<a href="lost_password.php"><div class="login-obr12"></div></a>
<div class="login-obr13"></div>
<a href="register.php"><div class="login-obr14"></div></a>
</div>
</form>
[HTML] pobierz, plaintext 

WTF? Na co tak?

[HTML] pobierz, plaintext 
<ul>
                <li><a href="home" onmouseover='obraz1.src="img/_09.gif"' onmouseout='obraz1.src="images_zew/_27.gif"'><img id="obraz1" src="images_zew/_27.gif"  style="display: block;" alt="obrazek"></a></li>
                <li><a href="register" onmouseover='obraz2.src="img/_11.gif"' onmouseout='obraz2.src="images_zew/_29.gif"'><img id="obraz2" src="images_zew/_29.gif"  style="display: block;" alt="obrazek"></a></li>
                <li><a href="screen" onmouseover='obraz3.src="img/_12.gif"' onmouseout='obraz3.src="images_zew/_30.gif"'><img id="obraz3" src="images_zew/_30.gif"  style="display: block;" alt="obrazek"></a></li>
                <li><a href="/forum" onmouseover='obraz4.src="img/_13.gif"' onmouseout='obraz4.src="images_zew/_31.gif"'><img id="obraz4" src="images_zew/_31.gif"  style="display: block;" alt="obrazek"></a></li>
                <li><a href="partners" onmouseover='obraz5.src="img/_14.gif"' onmouseout='obraz5.src="images_zew/_32.gif"'><img id="obraz5" src="images_zew/_32.gif"  style="display: block;" alt="obrazek"></a></li>
				<li><a href="tutorial" onmouseover='obraz6.src="img/_15.gif"' onmouseout='obraz6.src="images_zew/_33.gif"'><img id="obraz6" src="images_zew/_33.gif"  style="display: block;" alt="obrazek"></a></li>
				<li><a href="contact" onmouseover='obraz7.src="img/_16.gif"' onmouseout='obraz7.src="images_zew/_34.gif"'><img id="obraz7" src="images_zew/_34.gif"  style="display: block;" alt="obrazek"></a></li>
 
				</ul>
[HTML] pobierz, plaintext 

A za to powinieneś dostać batów, żebyś raz na zawsze zapamiętał, że:

• inline JS się nie powinno używać
• obrazki są tu całkowicie zbędne

[HTML] pobierz, plaintext 
<font color="lime">Miejsce</font>
[HTML] pobierz, plaintext 

To strona robiona 10 lat temu?

[HTML] pobierz, plaintext 
<script type="text/javascript">
 
  var _gaq = _gaq || [];
  _gaq.push(['_setAccount', 'UA-11794205-3']);
  _gaq.push(['_trackPageview']);
 
  (function() {
    var ga = document.createElement('script'); ga.type = 'text/javascript'; ga.async = true;
    ga.src = ('https:' == document.location.protocol ? 'https://ssl' : 'http://www') + '.google-analytics.com/ga.js';
    var s = document.getElementsByTagName('script')[0]; s.parentNode.insertBefore(ga, s);
  })();
 
</script>
[HTML] pobierz, plaintext 

Statystyki ZAWSZE powinno się dawać pod koniec strony, a nie na początku.

Już na koniec puszczam pod walidator...

Errors found while checking this document as HTML 4.01 Strict!Result: 45 Errors, 41 warning(s)

Po co używasz strict, skoro kodowi do tego daleko...?

Teraz grafika:

Number of inline elements:
53 (540 395 bytes)

Nie uważasz, że to trochę za dużo?

Jeśli chodzi o grafikę - na co sklejasz tyle zawartości z tłami, skoro mogą one być bez problemu zrenderdowane przez przeglądarkę? Zmniejszyłbyś rozmiar grafik. I zmieniłbym miejsce, w którym masz menu - dopiero po jakimś czasie załapałem, że tam w ogóle ono jest.

Reasumując:

• Kod: 4/10
• grafika: 7.5/10

PS. W screenach nic nie ma. (IMG:style_emoticons/default/tongue.gif)