Code & Design reviews - Daniquedejong/visual-thinking GitHub Wiki

Code/design -Presentatie

Minicursussen

Badr

Tijdens ons gesprek met opdrachtgever Charley heb ik de tijd genomen om het nieuwe ontwerp van de minicursussen pagina te presenteren.

Ik legde duidelijk uit dat deze pagina was met JavaScript gemaakt, zodat je door slides kon swipen. Zonder JavaScript was de pagina niet bruikbaar. Daarom heb ik deze pagina geherstructureerd en dynamisch gemaakt, wat betekende dat ik dezelfde styling niet kon behouden en een nieuw ontwerp moest maken, dat ik snel in de code heb geïmplementeerd aangezien de website al live was, deze pagina was natuurlijk beter voor de eindgebruiker en ook responsive en beter dan de vorige oude versie en consistent met de andere pagina's.

Heel simpel ik heb tegen Charley gezegd dat 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, en de oude versie niet echt consistent was met de website.

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.

Minicursussen nieuwe ontwerp

Pull request review

Danique

Op woensdag 24 april heb ik 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.

Filter

Badr

Deze week, in week 16, heb ik mijn studiegenoot Danique eindelijk mijn filter gepresenteerd. Ze was erg blij omdat het filter goed werkt en precies doet wat het moet doen. Tijdens onze sessie heb ik haar mijn code laten zien en uitgelegd hoe het werkt, zodat ze deze mogelijk ook kan gebruiken voor haar eigen project. Ze vond de code zeer nuttig en overweegt om deze in haar werk te integreren.

Daarnaast heb ik uitgebreid uitgelegd waarom ik heb gekozen voor een server-side filter in plaats van een client-side versie, die ik eerst had gemaakt. Ik legde uit dat de server-side filter efficiënter is omdat het de belasting op de client vermindert en de prestaties verbetert, vooral bij grotere datasets. Dit komt omdat de data op de server wordt verwerkt en gefilterd voordat het naar de client wordt gestuurd, waardoor de client minder werk hoeft te doen en de gebruiker een soepelere ervaring heeft.