De website op zich is mooi, goeie combinatie van frisse en donkere kleuren, dus op dat vlak zit alles wel redelijk goed.
De code is echter niet zo goed, het is een begin om met divs te werken maar je past ze niet helemaal correct toe.
- Je menu is niet opgebouwd door middel van een list maar gewoon met een hoop a's achtereen.
Dit is de manier waarop het zou moeten.
- Je gebruikt een pak teveel divs. Normaal zijn divs containers voor samenhorende elementen en daarbinnen kan je dan via alle andere elementen alles ordenen, bij jouw website designs zie ik bijvoorbeeld voor alle thumbs één div met daarin nog eens voor type, client en what een extra div. Dit zou in mijn ogen een div(²) moeten zijn met een image en 3 spans.
Die div line kan je simpelweg vervangen door een background op die div(²) toe te passen en die background helemaal onderaan te zetten zonder repeat-y.
- Webtop.gif, .. zou niet als img in je site moeten staan. Het zou een h2 od moeten zijn met dat als achtergrond. Als volgt:
<h2 style="width: height: background: "><span style="display: none">Websites</span></h2>
Die span is voor het geval je stylesheet niet geladen wordt en voor google om te begrijpen welke pagina hij leest. (en nog wel andere redenen).
Ziezo, dat zijn wellicht de meeste dingen waar je op dit moment rekening mee moet houden.
In een notendop:
- Images die niet tot de content behoren moeten altijd ergens als achtergrond geïntegreerd worden (daarmee refereer ik naar de h2 in de bovenstaande uitleg).
- Menu's in xHTML moeten met behulp van een list opgebouwd worden
- Divs zijn containers voor andere elementen, niet álles in divs gooien.
