Daily Checkout 6 december - DivaniNL/i-love-web GitHub Wiki

6-12-24 Code/Design-review over de code van de User Interaction

Vandaag heb ik samen met Branco en Clarice feeback gegeven op de code en de manier van feedback/feedforward.

De code moesten we op de volgende elementen beoordelen:

Controleer het JavaScript-bestand en beoordeel of er duidelijke code-comments zijn toegevoegd, zoals beschreven in het kopje Van Comments naar code. Zijn de comments begrijpelijk en helpen ze om de logica van de code te volgen?
Bekijk de aangemaakte variabelen in het JavaScript-bestand. Geven de namen van de variabelen een goed beeld van wat erin wordt opgeslagen?
Controleer of er styling (CSS) wordt aangepast in het JavaScript-bestand. Indien dit het geval is, maak dan een issue aan om HTML, CSS en JavaScript van elkaar te scheiden in aparte bestanden. HTML moet verantwoordelijk zijn voor de structuur en inhoud van de pagina, CSS voor de opmaak en JavaScript voor de interactiviteit. Bespreek met het team hoe je dit het beste kunt oplossen, bijvoorbeeld door gebruik te maken van de classList.toggle() methode.

Het design moesten we controleren op de volgende punten:

Analyseer met elkaar hoe feedforward is toegepast in het ontwerp. Begrijpt de gebruiker meteen wat er zal gebeuren bij interactie met het element? Bespreek met de groep of er visuele of tekstuele verbeteringen mogelijk zijn om dit te verduidelijken.
Beoordeel hoe feedback is verwerkt in de interactie. Is het voor de gebruiker duidelijk wat er gebeurt nadat een interactie heeft plaatsgevonden? Zijn de signalen die feedback geven (bijvoorbeeld kleurveranderingen, meldingen of animaties) logisch?
Maak een issue aan hoe de interactie nog vetter, functioneler of toegankelijker gemaakt kan worden. Gebruik hiervoor jullie verbeeldingskracht. Het mag out-of-the-box zijn!

Wij hebben Jules, Saif en Karima beoordeeld: