Skip to content
RickVellingaa edited this page Jan 26, 2023 · 1 revision

Tijdens deze deeltaak heb ik in de eerste en tweede code reviews gehouden met medestudenten, hierin heb ik goede feedback gekregen over mijn designs en code. Zo heb ik bijvoorbeeld als feedback over mijn code gekregen dat ik in plaats van een div ook een beter HTML semantisch element kan gaan gebruiken zoals een main op een bepaalde plek. Op mijn design heb ik als feedback gekregen dat het misschien mooier zal zijn als ik elementen meer met elkaar align zodat de witruimtes hetzelfde zijn en het mooier wordt.

Ook had ik tijdens de eerste sprint een color contrast test gedaan om te kijken hoe goed het contrast is van Funda, er kwam uit dat het contrast niet echt goed is, waardoor het slecht leesbaar is voor vele mensen. Dat was ook al bekend bij de opdrachtgever maar hij wou dat wij ons alsnog aan de huisstijl houden, dus dat heb ik ook gedaan.

Contrast\

Ook heb ik tijdens de sprints elke keer mijn HTML en CSS getest om te kijken of ik niet veel fouten aan het maken was terwijl ik aan het coderen was, om dit te doen heb ik een HTML en CSS validator te gebruiken. Hieruit was gekomen dat ik 2 fouten in de HTML namelijk dat ik een paar lege alt teksten had in een img element, de CSS validator was goed, want daar kwamen geen fouten uit.

In de laatste sprint, voordat ik mijn pagina op ging leveren aan de opdrachtgever, heb ik nog een lighthouse test gedaan, hier kwam uit dat het kleuren contrast niet goed was/niet goed zichtbaar is voor mensen met slechtere ogen. Ook kwam eruit dat de iframe geen title had.

Lighthouse

Clone this wiki locally