Archief - CLANSITE: Toysoldiers

Hoi daar
De volgende site heb ik gemaakt voor mijn clan, ik moet er wel bij zeggen dat het de eerste keer is dat ik een website maak in PHP & MySQL

http://tsi.e4media.be/ * Met dank aan www.e4media.be

Ik hoop dat het jullie bevalt...

eerste kleine dingetjes die mij opvallen :
geen mouseovers
en geen fout-uitleg bij inloggen. (dat is nie echt een must maar toch gebruiksvriendelijk)

bt belgium zei:

eerste kleine dingetjes die mij opvallen :
geen mouseovers
en geen fout-uitleg bij inloggen. (dat is nie echt een must maar toch gebruiksvriendelijk)

Klik om te vergroten...

Correct, dat laatste ga ik even aanpassen.

About the mouseovers.. Je bedoelt bij de menu?

Zonder naar de layout te zien >> WTF is dat met u code man ?

verschillende doctypes
verkeerd geplaatste tags
...
Tes massas onduidelijk

Zwarte letters op donkergrijze achtergrond is met momenten zeer onleesbaar

/edit
Na de comment van Bunz gelezen te hebben

IDD! uw code

Waarschijnlijk volledige pagina's ge-include ipv enkele stukken code

Bunz zei:

Zonder naar de layout te zien >> WTF is dat met u code man ?

verschillende doctypes
verkeerd geplaatste tags
...
Tes massas onduidelijk

Klik om te vergroten...

Ik ben dan ook geen pro, ik zal eraan werken ..

Eerste Indruk

mooi design op het eerste zicht. Kleurenschema zit wel goed.
Code kan ik nog niet zien maar daar kom ik wel later op terug.

Design

-Oranje en grijs past mooi bij elkaar, alleen in de nieuwsposts.
-Die donkergrijze achtergrond zorgt ervoor dat je tekst zelf moeilijk leest, maak die achtergrond een beetje lichter en het is al een stuk beter.
-Rollovers bij het menu ontbreken: Dit zorgt er voor dat het moeilijk is om te zien welke link je aanduid. Een simpele kleurverandering als je je muis erover beweegt verhelpt dit probleem.
- Die spacing tussen je menukolom, content area en het login gedeelte vind ik niet zo geslaagd (dit is zuiver mijn persoonlijke mening), het komt een beetje over als een niet samenhangend geheel.

Code

- Doctypes: Zoals hierboven gezegd, je gebruikt verschillende doctypes in 1 html bestand. Verander dat.
-Rommelige code: Zorg voor wat structuur in je code, anders is het moeilijker om fouten op te sporen.
- Javascript: Je gebruikt in je code 2 keer hetzelfde script. Laat dat eruit en doe dit louter met je CSS-code. Je bereikt hetzelfde resultaat en het is nog makkelijker ook.
- Je maakt in 1 pagina eigelijk 5 verschillende pagina's! 5 keer open en sluit je een html tag: Doet dit slechts 1maal en maak gebruik van div's voor je layout.
Tabellen zijn voor gegevens, div's voor je layout (al veel gezegd maar herhalen kan nooit kwaad )

Conclusie

Mooie site met enkele minpunten en een ZEER rommelige code.
Door de code brengt het mijn eindbeoordeling een stuk naar beneden, maar die code kan je gemakkelijk opnieuw schrijven. Veel succes! Je bent iig op de goeie weg.

Score

6/10

Design

-Rollovers bij het menu ontbreken: Dit zorgt er voor dat het moeilijk is om te

Klik om te vergroten...

Rollovers zitten erin

raintjah` zei:

Rollovers zitten erin

Klik om te vergroten...

ok, nu ziet ge tenminste op welke link ge klikt

Featherfoot zei:

ok, nu ziet ge tenminste op welke link ge klikt

Klik om te vergroten...

Yep da's wel waar.
Kga der mee stoppen voor vandaag ;D Ik word onderhand al dul van al die code
Thx voor alle advies mensen!!!

UPDATE

**
Login is beetje verbeterd
**

-font op navigatie knoppen is echt niet mooi
-rollovers ervan ook niet (ik hou niet van het ingedrukte effectje)
-de header is vrij leeg, mist nog een effectje (screen, tekst, ...)
-er zit precies een blur op dat oranje pixelfontje
-results: maak het groen als ge wint, rood als ge ... dan is het tenminste duidelijk wie gewonne is
-kzou de oranje image die je gebruikt al background in de nieuwsitems etc iets anders maken, een lichtere tint dan de header
-tis niet direct duidelijk dat het een cs clan is
-over de code ga ik het niet hebben....

Ik versta echt nie oe dat gij u code hebt geschreven oor verschillende doctypes en zelf verschillende html tags alsof dat ge al u paginas op één pagina zet ....
Begin gewoon opnieuw met u code u img's ebde al enzo gewoon code opnieuw schrijven

Da is niet moeilijk zu die code zo fucked up schrijven....

is standaard fout numero uno van iemand die snel een tut neemt om te zien hoe includes werken.

heeft dus een index met al zijn doctypes in...
de pagina die geïnclude wordt heeft ook een doctype (ahja want zo schotelt dreamweaver of een andere paket hem voor als hij een nieuwe site maakt)

Idem dito behaviors genomen van Dreamweaver (niet dat, dat niet mag natuurlijk) maar die zijn veel te "zwaar" gescript voor hetgeen het eigenlijk maar is...




Website zelf nu

Clan? oké ja en welk game, da moet da uitroepen, of toch op zijn minst iets op de eerste pagina als "a counter-strike clan" "a multi-gaming clan"!

ben niet zo voor het "leeglaten" tussen de drie pannels... Da breekt zo de site los, de structuur die je wil creëren valt daardoor direct, surtout met zo'n contrastrijke bgr...

je newsposts hebben last van het "addslashes" command in php (zie: Place: Luna\'s Ijstadion, Lommel) waarschijnlijk, dit kan opgelost worden (check google)

vind typografie van broodtekst heel zwak, zit geen consistentie in... (zie hoe archive verschilt van achievements op uitlijning) Hoe in nieuwsposts dominantie is van bold en underline en op andere pagina's niet, hoe je niet duidelijk verschil ziet van titels en tekst,...

header vind ik behoorlijk, mist misschien nog een detail ofzo...
anders ook behoorlijk en degelijk kleurgebruik

qua coding ga ik niet verder uitwijden op wat al gezegd is

conclusie
voeg aan de poll optie "mja, maar nog veel werk aan" en ik vote daarop!

Ik heb geprobeerd de code IETS te verbeteren. Het script wordt niet telkens herladen, en alle head, body en html tags ook

NeoNeke zei:

je newsposts hebben last van het "addslashes" command in php (zie: Place: Luna\'s Ijstadion, Lommel) waarschijnlijk, dit kan opgelost worden (check google)

Klik om te vergroten...

Zo, dat is opgelost

doe op het einde van de dag 1 post waarin da ge alles opsomt wat ge hebt opgelost...

DieselPower™ zei:

doe op het einde van de dag 1 post waarin da ge alles opsomt wat ge hebt opgelost...

Klik om te vergroten...

Done

Ik zou ten eerste die smilie's uit je main posts halen, geeft een soort kiddo effect. En als je wilt dat je site serieus genomen wordt door sponsors ik weet het niet, 't geeft mij een vrij amateuristisch gevoel.

De kleuren zijn wel OK, maar t'is TEVEEL van hetzelfde grijs, probeer hier wat mee te spelen in tinten. Ook dat je middelste paneel zo "los" lijkt te hangen van links en rechts stoort me wel.

Voor de rest vind ik het wel "ok". Toch een van de beter clansites tot nu toe dat we hier te zien krijgen, want het kan hier nog veeeel erger dan dat .

Die zwarte tekst op dat grijs: niet leesbaar (reeds vermeld maar aangezien er nog geen verandering is...)