Code Review van de Eendtjes (Clarice, Amber & Julia)

[julia-stevens]

1. Er zijn een aantal errors volgens de w3c validator: https://validator.w3.org/nu/?doc=https%3A%2F%2Fdebosos.github.io%2FGeef-een-tien-ofzo%2F

   • Bijvoorbeeld: gebruik geen figcaption in een div, zie: https://developer.mozilla.org/en-US/docs/Web/HTML/Element

2. Jullie gebruiker header, main, section

   • section moeten altijd een heading hebben: https://developer.mozilla.org/en-US/docs/Web/HTML/Element/section https://github.com/DeBosOs/Geef-een-tien-ofzo/blob/main/index.html#L18-L31
   • figcaption mag alleen een kind zijn van een figure https://github.com/DeBosOs/Geef-een-tien-ofzo/blob/main/index.html#L34-L37

3. jullie gebruiken comments, misschien zou je bij een aantal onderdelen wat meer uitleg kunnen geven, bijvoorbeeld bij een ingewikkelder element

4. jullie hebben een uitgebreid grid dat helemaal responsive werkt

5. in de browser werkt alles heel goed, (ook in safari) ook bij het resizen van de browser, in de mobile versie werk de hover functie bij een klik, maar daar blijft hij dan ook op staan, misschien kunnen jullie hier nog wat mee. Dit geldt ook voor het logo, dit verandert van kleur bij een klik, maar blijft hier dan ook staan. Verder is alles wel repsonsive, en kun je alles goed bekijken.

6. misschien kunnen jullie nog iets met de vogeltjes? Bijvoorbeeld dat bij een hover of klik een vogeltje even uit de hoek komt kijken

[MarsGotBars]

https://github.com/MarsGotBars/Geef-een-tien-ofzo/commit/d85b6d90d2723b9dab8bc9e8cdf69d540ff2bfef punt 2.2 aangepast, 2.1 zullen wij onthouden voor de volgende keer, anders zou dit een redelijke code restructure introduceren.