joordy / browser-technologies

Hi there 👋

My open GitHub stats:

browser-technologies's Issues

Code Review w2

De website werkt prima met alleen HTML, met CSS erbij ziet het er een stuk fijner uit en werkt het daardoor ook fijner.
De core functionaliteit is goed uitgewerkt en werkt prima.
HTML ziet er goed en semantisch uit!
Er is nog geen feature detection toegepast.

Review

Core functionaliteit

De core functionaliteit van je app is goed uitgelegd en is bijna volledig werkzaam in de live demo. Alleen het laatste stukje van de flow, het bestellen van de opgeslagen t-shirts, moet nog worden toegevoegd.

Pleasurable

Volgens je README wil je de app pleasurable maken door een live preview te tonen van veranderingen aan het shirt en wil je het mogelijk maken om een shirt te verwijderen met swipe gestures. Dat lijken mij prima enhancements om de ervaring voor de gebruiker te verbeteren. De features zijn nog niet verwerkt, maar dat is logisch gezien we nog maar net bezig zijn.

Web features

Hier wil je gebruik maken van de Storage API, Touch events en Validity. De features zijn nog niet verwerkt, maar dat is logisch gezien we nog maar net bezig zijn.

Progressive Enhancement

Je hebt je demo opgebouwd vanaf HTML en hebt nu CSS toegevoegd. De core functionaliteit van je app is ook bruikbaar wanneer CSS is uitgeschakeld. JavaScript enhancements nog niet aan toegekomen. In je CSS kun je nog feature detection toevoegen, zodat je app ook werkt in browsers die bijvoorbeeld geen flex of grid ondersteunen.

HTML

De HTML lijkt valide en de app ziet er netjes uit!
Kwam nog een kleine Z-index issue tegen, zie: