fdnd-agency / deloitte

2 stars 1 forks source link

Peer feedback Remy #61

Open Remy2072 opened 1 month ago

Remy2072 commented 1 month ago

Readme

Opdrachtgever
Als opdrachtgever of niet-programmeur begrijp ik de inhoud van deze readme niet meteen. Er staan wel codefragmenten in, maar ik zou niet weten wat ik daarmee moet doen of waar ik die zou moeten invoegen. Zelfs als de opdrachtgever zou weten hoe dat moet, zou hij/zij waarschijnlijk een nieuw project starten in plaats van bestaande code aan te passen.

Frontender
Als frontender zou ik wel begrijpen wat de readme uitlegt, maar het is niet relevant voor dit specifieke project. De readme legt alleen uit hoe je een nieuw project opzet en een live server start, maar biedt geen informatie over het project zelf.

Testen

Performance

Login pagina
De performance van de loginpagina was niet optimaal, met een score van 76. Dit komt voornamelijk door de lange laadtijd van een afbeelding, die 4,5 seconden duurde. Probeer de afbeelding kleiner te maken om dit te verbeteren.

Hoofdpagina
Op de hoofdpagina speelt hetzelfde probleem: de afbeeldingen hebben niet het juiste formaat en de animaties zijn te zwaar, wat de website vertraagt. Als iemand een ouder apparaat gebruikt, kan dit problemen veroorzaken.

Toegankelijkheid
De toegankelijkheidstest was over het algemeen goed voor beide pagina's, maar de loginpagina mist een paginatitel.

Responsiveness

Login pagina
De loginpagina is niet responsive. Dit kan eenvoudig worden opgelost door bijvoorbeeld gebruik te maken van display: flex, zodat de afbeelding en inlogvelden netjes onder elkaar worden weergegeven op kleinere schermen.

Hoofdpagina
De hoofdpagina schaalt wel mee, maar is niet echt responsive te noemen. De tekst overlapt en alles wordt te klein en opeengepakt, waardoor het niet prettig leesbaar is op kleinere schermen.