[review] www.vleesenbeekbouw.nl

Hij bedoelt dit stukje JavaScript:

function swapFields()
{
	if(document.getElementById("particulier").style.visibility == 'hidden')
	{
		document.getElementById("particulier").style.visibility = 'visible';
		document.getElementById("particulier").style.height = 'auto';
		document.getElementById("particulier").style.display = 'block';
		document.getElementById("zakelijk").style.visibility = 'hidden';
		document.getElementById("zakelijk").style.height = 0;
		document.getElementById("zakelijk").style.display = 'none';
		document.form.soort_z.checked = false;
		document.form.soort_p.checked = true;
	}
	else
	{
		document.getElementById("zakelijk").style.visibility = 'visible';
		document.getElementById("zakelijk").style.height = 'auto';
		document.getElementById("zakelijk").style.display = 'block';
		document.getElementById("particulier").style.visibility = 'hidden';
		document.getElementById("particulier").style.height = 0;
		document.getElementById("particulier").style.display = 'none';
		document.form.soort_z.checked = true;
		document.form.soort_p.checked = false;
	}
}

[edit]Aanvulling: het is niet echt efficiënt om elke keer document.getElementById aan te roepen. Je kunt hem beter 1 keer aanroepen en het resultaat in een var stoppen.[/edit]

Oké! Ja, als de opdrachtgever met zo'n idee komt is dat prima uit te voeren nietwaar? ;)

@ jan: pagina valideert nu vlekkeloos. ;)

bart van der veen schreef op 13.09.2007 22:33

heeft er niemand even naar de code gekeken,
dit moet wel ander kunnen.
<?
.....
?>

@jan, het is 80 dezelfde error dankzij bovenstaande code.

Had ook mijn post even goed gelezen, ik zei al dat het herhalen van een afbeelding in CSS (repeat) veel sneller/ effectiever is dan een afbeelding tig keer herhalen.
Het was me al opgevallen zonder in de broncode te kijken, omdat je gewoon meteen ziet dat de afbeeldingen niet in 1 keer geladen zijn/ worden.

Offtopic en hopelijk niet relevant:

Welkom op de website van

Ik moet de eerste website nog tegenkomen waar ik niet welkom ben en tevens ligt het redelijk voor de hand dat ik op een website beland wanneer ik met m'n browser een beetje aan het rondsurfen ben. Zowel 'welkom' als 'website' zeggen dus helemaal niks.

Ons bedrijf is gespecialiseerd in het bouwen van muren met kalkzandstenen

Dit is wat ik horen wil! Met 1 korte krachtige zin wordt mij verteld wat het bedrijf in huis heeft. In een fractie van een seconde beslis ik dus of ik hier aan het juiste adres ben of juist niet. Dat is dus heel wat anders dan de kansloze openingszin 'Welkom op de website van ...'.

Leuke simpele site. Je kan nog wel wat verbeteren, maar het ziet er al goed uit.
Ik zou zelf de padding wat verhogen, zodat er meer ruimte is tussen de tekst en de rand. Ook de line-height heel iets. Dit leest prettiger. Je moet ervoor zorgen dat tekst prettig leesbaar is, anders haakt het grootste gedeelte van de bezoekers af.

Ik zou de css in een aparte stylesheet stoppen en niet bij elke pagina er hard in zetten.

En zorg ervoor dat je in het menu kan zien op welke pagina je je bevindt

tis volgensmij trouwens 'Built by' en niet 'Build by'. Past tense and all.

@ Niek, bedankt voor de feedback. Ik heb het allemaal aangepast, maar ik weet het nog niet zeker van het menu. Of ik het zo wel goed vorm gegeven heb. Misschien hebben jullie nog wat ideeën?

Kan idd beter. Als je deze style wilt aanhouden, zou ik sowieso die dubbele lijnen wijzigen in enkele lijnen

Niek schreef op 14.09.2007 11:11

Kan idd beter. Als je deze style wilt aanhouden, zou ik sowieso die dubbele lijnen wijzigen in enkele lijnen

Zo?

Ik zie dat je wel wat met die afbeeldingen van die randen hebt gedaan, maar eigenlijk nog niet zo efficiënt mogelijk, je hebt elke keer een "kleine steen, tussenruimte, grote steen, tussenruimte" dit patroon herhaalt zich steeds, als je een div maakt van de juiste breedte, en je geeft dit als achtergrond en geeft er bij aan (allemaal in css) dat die repeat-x (horizontaal herhalen) moet doen, en hezelfde onderaan, dan hoeft je pagina alleen maar 1 keer de afbeelding/ het herhalende deel van het patroon te laden en voor de rest wordt het door CSS herhaald.
Dat is de snelste en efficiëntste methode.