damianr2004 / look-and-feel-corporate-identity Goto Github PK

Het is semantischer om van deze button een input te maken met als type="submit"

Vergeet je input niet te koppelen met je label, je mist een name="naam"

https://developer.mozilla.org/en-US/docs/Web/HTML/Element/input/submit

Je kan eventueel border-bottom gebruiken voor beide sections gebruiken. Zo voorkom je dat je meer code schrijft om vervolgens de instelling die je eerder hebt geschreven uitschakelt. Zo vereenvoudig je jouw code en schrijf je hiermee efficiënter CSS.

Hi Damian (:

Ik zie dat je gekozen hebt om de class namen in het Nederlands te doen, je zou kunnen overwegen om Engels te gebruiken.
Je zou kunnen kijken/navragen welke manier jouw opdrachtgever gebruikt. Als alles in het NL is, is het natuurlijk beter om in jouw code ook NL te gebruiken maar als er ook internationale collega's aan het project werken kan Engels een betere optie zijn!

Je mag een h1 maar 1 keer gebruiken. Gebruik de h1 voor de belangrijkste titel op de pagina. De rest van de header mogen h2,h3 ect worden.

Top dat je gebruik maakt van custom properties om een dark mode toe te voegen aan je site. Check of je een semantische naamgeving kan verzinnen. Zo voorkom je dat wit zwart word en zwart wit word gedefineerd.

De annuleer button werkt nog niet

Ik vind dat je code er netjes en overzichtelijk uitziet, lekker bezig!💪

De mediaquerys kunnen beter worden geschreven, de gap en padding is bijvoorbeeld dubbel.

Maak het duidelijker dat je mensen kunt toevoegen aan de lijsten