-
Notifications
You must be signed in to change notification settings - Fork 0
Code & Design reviews
Tijdens ons gesprek met opdrachtgever Charley heb ik de tijd genomen om het nieuwe ontwerp van de minicursussenpagina te presenteren. Ik legde duidelijk uit waarom ik het oude ontwerp had gewijzigd: de oude versie gebruikte JavaScript, waardoor de pagina onbruikbaar was wanneer JavaScript was uitgeschakeld, wat nadelig was voor de eindgebruiker.
Het nieuwe ontwerp dat ik heb gemaakt, is veel gebruiksvriendelijker. Charley vond dit nieuwe ontwerp goed en gaf toestemming om het te gebruiken in plaats van het oude.
Op woensdag 24 april heb ik (Danique) samen met Joost gezeten om te kijken naar een pull request die ik had ingediend. Deze pull request ontbrak nog aan een aantal dingen, waaronder het benoemen van querys en arrays in een html file.
Eerst:
const methods = data.method[0];
Nu:
const method = data.method;
Tot slot hebben we ook de file namen aangepast, zoals de files in het queries mapje. Deze heetten namelijk: url.js & url2.js Dit is natuurlijk niet best. Zo ziet de nieuwe naam eruit.