[www] Serwis dla gracza Opcje

[c2h5oh]

http://giery.eu

Czekam na uwagi :-)

[gszesiek]

jedna uwaga, bo nie chce mi się czekać, aż się ściągnie całość ->
zrób miniaturki obrazków i z nich linki, bo użycie dużych strasznie przedłuża oczekiwanie (jak ktoś ma wolne łącze - jak ja)
pozdrawiam

validator - użyj
poziome menu - nieczytelne
rozkład - (jak dla mnie, a nie oglądam takich witryn) mało czytelne, za dużo info/piksel^2
banner - nie podoba mi się

--------------
przepraszam, że piszę tylko o tym co mi się nie podoba (bo jest sporo rzeczy, które mi się podobają ), ale chyba tego oczekujesz

Ten post edytował gszesiek 2.11.2006, 00:20:07

[c2h5oh]

O tym jednym wiem. Szukam bledu w generatorze miniaturek juz ktorys dzien Inne uwagi?

[Adrian Staniszew...]

Prezentuje sie ladnie, grafika prosta lecz ladna i nie meczy oka, kolory troszke malo przemyslane - lekko za wyblakłe.

Logo.. w sumie to nie ma zadnego..a moze sa.. nawet 3

Duzo reklam google.. moze za duzo ?

Nazwa.. mimo wszystko nie pasuje do serwisu, bo prezentuje troszke wyzszy poziom niz by o tym swiadczyla.

Od strony programistycznej tez jest troszke niedorobek.. chocby kilka sposobow uzycia mod revrite - raz katalogujecie, raz przerabiacie do html, a raz uzywacie zwyklych.. nieladnie to wyglada

ogolem dosc duzo niedorobek, ale wyglada porzadnie i trzyma poziom

moja ocena: 4/6

Ten post edytował Adrian Staniszewski 2.11.2006, 00:06:17

[Kreton]

[HTML] pobierz, plaintext 
<div class="menubar">
<div style="float:left; width: 100%;">
<table border="0" cellpadding="0" cellspacing="0" width="100%">[/b]
<tr>
<td><a href="http://giery.eu/PlayStation-3/" class="menups3" ></a></td>
		[...]
<td><a href="http://giery.eu/Inne/" class="menuinne" ></a></td>
[HTML] pobierz, plaintext 

Hmm, co to tu robi ? Z kąd te tabele. W zasadzie to jest menu, wiec nie wiem po co te tabele. Dodatkowo roisz <div style=""> Mamy przecież style zewnętrzne.

[HTML] pobierz, plaintext 
div class="moduletablemenu">
 
<table width="100%" border="0" cellpadding="0" cellspacing="0">
<tr align="left"><td><a href="/" class="mainlevelglowne" id="active_menuglowne">Strona główna</a></td></tr>
<tr align="left"><td><a href="http://giery.eu/Newsy/" class="mainlevelglowne" >Newsy</a></td></tr>
<tr align="left"><td><a href="http://giery.eu/Zapowiedzi/" class="mainlevelglowne" >Zapowiedzi</a></td></tr>
<tr align="left"><td><a href="http://giery.eu/Recenzje/" class="mainlevelglowne" >Recenzje</a></td></tr>
<tr align="left"><td><a href="http://giery.eu/Sprzet/" class="mainlevelglowne" >Sprzęt</a></td></tr>
<tr align="left"><td><a href="http://giery.eu/Publicystyka/" class="mainlevelglowne" >Publicystyka</a></td></tr>
</table>		</div>
[HTML] pobierz, plaintext 

WTF ? Znowu, tabele, tutaj to już centralnie menu, które można, ba, trzeba zamienić na listy wypunktowane. Tabele wywalić ! A styli do linków nie dodawaj osobne, prezcież można prościej : div.moduletablemenu a { tutaj jakieś style;} Dalej to samo, w taki zły sposób robisz kolejne menu.

Dalej, trzeba rozróżnić kiedy dajemy class a kiedy id. Id do obiektu dajemy tylko wtedy kiedy wiemy, że pojawi się tylko raz na stornie. Np: wiesz że kontener center albo main nie pojawi się n-razy tylko 1 raz. Wtedy dajemy id. Class dalejmy jak wiemy że elementów będzie napewno więcej niż 1. Np ty masz te paskudne : tableki z linkami. Można użyć class bo jest ich więcej niz 1.

Ogólnie kod to niezła sieka. Storona długo się ładuje ze względu na tabelki, których są setki. A teraz niech, któraś okaże się nie zamknięta ( modyfikacje w przyszłości ). To życzę Powodzenia.

Niektóre obrazki u Ciebie to jakiś kosmos. Widziesz ten obrazek z helikopterem od razu w centrum. Waży ~55kB. Wow, szczególnie, że obrazek obok waży 15kB a jest takiej samej wielkości.

Grafika : 7/10 - ładnie się prezentuje.
Kod : Sieka

Ten post edytował Kreton 5.11.2006, 15:36:30