[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

[Kenobi]

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

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

[karol33]

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

[gigzorr]

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

Ten post edytował gigzorr 9.07.2010, 19:23:14

[Van Pytel]

Grafika ladna, nad reszta trzeba popracowac.

[Substr]

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.

[nmts]

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

[Neon12345]

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.

[Pilsneer]

Oba błędy zostały poprawione.

W kilku miejscach można tez załadować np kod HTML..

Nie rozumiem?

[cojack]

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

[Neon12345]

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

[cojack]

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

Ten post edytował cojack 10.07.2010, 08:17:51

[Neon12345]

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"

[conel]

@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

[Neon12345]

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 .

[gigzorr]

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

Ten post edytował gigzorr 10.07.2010, 10:55:24

[Pilsneer]

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

[maateusz]

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

[vtuner]

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.

Dzisiejszego wieczoru została wprowadzona opcja Własny Biznes, mogą pojawiać się drobne błędy ale o tym później

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)