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

Skala od 0/10

Mi się trochę nie podoba ramka tabel.
Ale nawet jest bardzo dobrze jak na taką grę ;]
8,5/10

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...

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

Teraz grafika:


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.

a mi sie podoba... kurcze kiedy sam sie naucze pisac takie skrypry...kawał dobrej roboty grafikę uważam że nie ma co dopieszcza bo zawsze moze trafic sie ktos kkomu sie nie spodoba

Kod mnie przeraził z poczatku(czemu wogole id nie uzywacie tylko same klasy?) , pare bledow do poprawienia http://validator.w3.org/check?uri=http%3A%...ine&group=0

6/10

Grafika ladna, nad reszta trzeba popracowac.

Grafika możne i ładna, ale kod pozwala mi na zarejestrowanie takiego samego nicka jak np administrator przez co zyskuje uprawnienia.. W kilku miejscach można tez załadować np kod HTML..

Nic nie skasowałem, no może poza swoim kontem które wcześniej założyłem dla testu.

Ustawiłem sobie unikatową pozycję 'Mistrz'. Poza tym wystarczy komuś podrzucić fake linka żeby wykonał akcję typu Zakup premium i wiele innych?

Robimy to we dwóch dlatego , jest tyle błędów. Osoby chętne do pomocy ze znajomością PHP , MYSQL , JS zapraszam pod GG: 4631946 (niewidoczny)

Tak więc wszystkie wymienione błędy zostały naprawione . Rejestracja na administratora jak i zmiana pozycji.

Oba błędy zostały poprawione.



Nie rozumiem?

Dlaczego mam wrażenie że skrypt to PHP Fusion?

nie mam pojęcia. Ale czy na skrypcie php-fusion , nie zamieścisz autorskiej gry?

Jeżeli to php-fusion to złamałeś warunki licencji ;]

Ej czy mi się wydaje czy ty jesteś totalnym dzieciakiem , który nie zna się wogól na programowaniu, a tym bardziej nie umie spojrzeć w źródło strony. Lub wogóle odróżnić CMS? Może forum.php.pl jest postawione na php-fusion "HAHAHA"

@Neon12345 Na chwilę obecną to Ty się tak zachowujesz.

Na pierwszy rzut oka widać, że to robota w stylu kopiuj->wklej + zero umiejętności. Layout wygląda "za ładnie" jak na resztę elementów (np. form rejestracji - jak można było nie uporządkować tych pół?). Pola logowania także kompletnie schrzanione. Lewa strona za szeroka jak na występującą tam treść. Podstrona "tutorial" brak polskich ogonków.

Jeśli zaś chodzi o kod, to nie ma sensu się wypowiadać, szkoda czasu. Zresztą kwiatki typu rejestracja na istniejącego użytkownika pokazuje dobitnie "profesjonalizm" autora.

Podsumowując: gotowy layout - zero modyfikacji, aby to jakoś wyglądało (boxów się nie chciało już zwęzić bo to oznacza edycje grafiki, nie?). Kod bez ładu i składu, byleby było.

0/10

Dobrze następna mądra wypowiedź , tak więc kod rzeczywiście jest namieszany ponieważ nie zdążyliśmy go poukładać jest nas dwóch i zajmujemy się pisaniem skryptów do gry bo uważam , że jednak to jest bardziej potrzebne niż kilka niedociągnięć css,html .

te inputy przy logowaniu to wam grafik robil?jesli tak to opierdziel go , bo zle wyciete , albo zrobcie przezroczystosc.

A do tego info o partnerach , ciekawe jak bedziesz mial ich 100 co zrobisz ?daj overflow: scroll; , i pomniejsz heighta dla mnie by to lepiej wygladalo tak.
Ten handball manager napis zrobilbym klikalny gdybym byl na waszym miejscu.
A te inputy przy rejestracji to zwykle marginesy im nadaj i bedzie ok
Formularz kontaktowy mozna niezle zaspamowac , na waszym miejscu sprawdzalbym chociaz ilosc znakow np. min 20 w polu text area albo cos w tym stylu.
Poco wam przy rejestracji moje dane osobowe mam na mysli imie/nazwisko ? Przeciez i tak tam nie wpisuje prawdziwych a innych to odpycha?
Wlasnie zmienilem avatar na tapete 2mb .png , a jest napisane tylko .jpg?ehm wlasnie wgralem shella , wogole danych nie sprawdzacie , ale idk jak sie nazywa was folder z avatarmi wiec dupa , radze to poprawic jak najszybciej.A co do forum to bym na waszym miejscu zmienil wyglad + zrobil przyjazne linki , odrazu byloby fajniej , jakies ciemne kolory na forum np. jakies fajne theme wgrajcie czy coś , bo tak jakoś na odpierdziel

Imię i Nazwisko którym będziesz posługiwał się w grze, nie wymagamy waszych prawdziwych danych, ma to być zmyślona nazwa.

A co do błędów postaramy je naprawić.

Dość przyjemnie to wygląda, jest pare niedociągnięć o których ktoś wspominał wyżej, ale generalnie jest ok.

Rozumiem Neon12345, że Ty uważasz się za profesjonalistę? Po co dodajesz coś do publicznego użytku, skoro sam tego dobrze nie przetestowałeś?

A tekst typu "jest nas dwóch" jest świetną wymówką (czujesz ironie, nie?)

Kod: 1/10 (bo jednak jako tako działa)
Grafika: 4/10 (brak polskich znaków to poważny błąd)

"POKIERUJ KARIERIA SZCZYPIORNISTY"

w http://handball-manager.pl/img/_10.gif

No totalny ze mnie dzieciuch

tylko ja mam tak zrytą przeglądarkę czy Wam też rollover w menu po lewej tak opóźnia?
co do kodu, rzeczywiście masakra, aż dziw, że się to nie wysypało ;D
poza tym pełno literówek etc.
ogólnie słabo, ale grafika przypadła mi do gustu, dam 4/10

&/10
Jakoś mi ten zaielony do niebieskiego nie pasuje

Mi rollover dziala normalnie, nie opoznia
Co do kodu sie nie wypowiadam, bo nie widzialem, a i specjalista w tej dziedzinie nie jestem, ale jesli chodzi o sama grafike to takze przypadla mi do gustu!
Za grafike daje 9/10.

@Pan ponizej.
Ja na FF mam wszystko OK - ciekawa sprawa
Dogram jeszcze chrome i opere to zobacze, bo jak narazie jestem swiezo po reinstalce systemu (moze dlatego jeszcze mi rollover smiga)

EDIT: Dogralem Opere i Chrome i tak:
- na Operze takze smigaja rollovery
- na Chrome mula i to strasznie! (a jednak nie, tylko przy pierwszej wizycie, moze swieza instalka chroma sie jeszcze nie ulozyla wygodnie w systemie, bo teraz smiga wszystko).

U mnie też się opóźnia. (Na Chrome'ie i FF)

A może by tak po prostu użyć preloadu albo CSS-sprites?