|
Huidige score: |
|
70 |
|
|
Een volkomen zinloos ontwerp zonder functie.
Ik ben eraan begonnen omdat ik wilde experimenteren met xhtml, en heb ook maar meteen de overstap gemaakt van html 4.01 transitional naar xhtml 1.0 strict. Nadat ik de opzet met div'jes klaar had en het zag er goed uit in FF, bekeek ik het in IE.. Precies hetzelfde! Haha niet verwacht, ik schrijf t succes toe aan xhtml strict.. 
Ik hoor graag jullie meningen over mijn werk en hoop dat er, ondanks mijn gebrek aan photoshop-talent, bruikbare kritieken komen |
|
Gepost op:
09 juli 2006 - 11:38 |
|
|
|

SEO guru

|
Het ziet er goed uit. Voor snel even ineen te boxen dan. Je code is proper, MAAR:
*Leer semantisch te scripten. Een titel geef je de h1 tag mee, een subtitel een h2 tag, enzovoort. Het komt niet alleen de seo ten goede, maar zal je bron korter en overzichtelijker maken.
Het is echt niet zo moeilijk om het te doen en heeft heel veel voordelen.
*Je menu schrijf je beter als een lijst. Mbv css kan je je menu dan stylen tot een blok, of wat je maar wenst. Zoek maar even op google, je vindt vele voorbeelden. Waarom hoor ik je denken? Eigelijk is je menu een lijst, opsomming van wat je site te bieden heeft. En het ziet er netter uit.
*Wat me ENORM tegenvalt is die table tag. Waarom staat die ertussen? Wil je het centreren? Dan bestaan er genoeg css manieren. Nu zit alles in 1 grote tabel. Je layout dus ook. En die heeft er niets mee te zien.
Voor de rest zou ik er graag nog een footertje bij zien, en zou je manier van slicen nog iets beter kunnen.
Als je deze afbeelding bekijkt:
http://runningr...enkant.gif
Waarom steek je deze erin? Mbv css kan je een lijntje van 1px horizontaal laten terugkomen.
Steek het geheel in een content div en geef aan wat de padding/margin moet zijn (niet met de tabel), en steek er een oranje bordertje rond. Alleen al zo zal de laadtijd van je pagina veel lager liggen.
Spijtig ook dat de tekst in de header als afbeelding staat. Zou een ideale H1 zijn, zo als titel.
Je bent goed op weg. Nu nog aanpassen wat ik hierboven vermeld en je krijgt een 10 voor de code, een 7.5 voor de layout.
*nog geen stem* |
|
|
|
Gepost op:
09 juli 2006 - 17:52 |
|
|
|
HTML ver gevorderde

|
Hé
Bedankt voor de reactie, ik zal even wat vragen van je beantwoorden, ik ga met de tips aan de slag maar niet nu want ik heb het druk..
De table staat er alleen in, omdat ik de div's van de content en het menu naast elkaar wil plaatsen, en gelijk de hele boel kan centreren (class="center", en in mijn stylesheet bij ".center" margin-left en margin-right op auto). Eigenlijk uit gemakzucht, ik ga het verbeteren . Alle margin's en paddings zijn verder via het stylesheet en div'jes geregeld.
De bovenkant.gif, is volgens mij niet na te maken met css? Er zit een subtiele gradient gan lichtgrijs naar wit in. Zonder dat is de pagina erg plat..
Manier van slicen? Ik slice niet. Ik schrijf de code zelf, en maak de individuele afbeeldingen met photoshop. Een hele pagina in photoshop maken krijg ik niet voor elkaar, ik ben niet zo handig met ps .
Binnen een paar dagen zal ik de verbeterde versie uploaden, nogmaals bedankt voor de kritieken! |
|
|
|
Gepost op:
09 juli 2006 - 19:21 |
|
|
|

SEO guru

|
Wat ik bedoelde was dat die grote afbeelding vervangen kan worden door 1 klein afbeeldingetje. Al bij al een stuk of 60px op 1px. Je kan dat als achtergrond doen met repeat-x. Versta je?
Slicen kan je ruim nemen hoor. Jou manier is 1 van de beste. Zo kan je ze ook nog optimaliseren naar grootte toe.
Ik vrees dat ik dit niet echt verder meer kan volgen, want morgen om 3 uur vertrek ik op reis voor 2 weken.
Succes ermee. |
|
|
|
Gepost op:
10 juli 2006 - 11:02 |
|
|
|
HTML beginner

|
het hoeft niet altijd speciaal te zijn om er goed uit te zien
een 8 |
|
|
|
Gepost op:
13 juli 2006 - 12:36 |
|
|
|
PHP ver gevorderde

|
Ik vind hem niet zo, die balkjes rechts kan je beter doen als je deze zonder verloop er mee doet, maar dat een mooier kopje. |
|
|
Posten van een reactie is niet mogelijk. |
Bekijk: |
|
|
W3 Validiteit: |
|
|
|