Clarice-COD / the-client-website

Ontwerp en maak een website voor een opdrachtgever en bespreek het resultaat tijdens de Sprint Review
https://clarice-cod.github.io/the-client-website/
MIT License
0 stars 0 forks source link

Code review - Toegankelijkheid - Nadia & Julia #19

Open julia-stevens opened 1 month ago

julia-stevens commented 1 month ago
    1. Je gebruikt de a tags op de juiste manier om te linken, veel hiervan hebben ook een img met daarbij een alt
  1. Je gebruikt hier een input, maar hier zou nog een label omheen moeten https://github.com/Clarice-COD/the-client-website/blob/ccbb1475d044445e3fa4385b0fa73c3ce8e6343b/index.html#L31
  2. Je hebt hier een h3 gebruikt, maar deze is in de content niet te zien, misschien kun je deze weghalen of verplaatsen, zodat het wel te zien is, daarmee heb je meer structuur in je heading elementen https://github.com/Clarice-COD/the-client-website/blob/ccbb1475d044445e3fa4385b0fa73c3ce8e6343b/index.html#L43
  3. Je article moet een heading hebben, je footer moet in de body en je gebruikt een spatie in de naam van de afbeelding, deze kun je beter scheiden met een streepje (-) of aan elkaar schrijven Zie hier de validator: https://validator.w3.org/nu/?doc=https%3A%2F%2Fclarice-cod.github.io%2Fthe-client-website%2F image
  4. Je gebruikt nav, header, main, footer let wel op dat footer ook in je body staat!
  5. Je lighthouse score is 96! Door de h3 weg te laten of te veranderen, heb je een betere score.
  6. Keyboard test ging goed, misschien is het qua volgorde wel logischer als je eerste naar de search gaat en daarna naar het menu. Misschien kun je van het e-mail adres in de footer nog een linkje maken
  7. Kleur contrast is goed
  8. Wanneer we inzoomen tot 200% blijft alles leesbaar, alleen het menu aan de zijkant kun je niet meer gebruiken. Misschien kun je hier iets van een overflow toevoegen?
  9. Misschien kun je van het menu een list maken, zodat een screen reader voorleest dat het om een lijst gaat en hoeveel items er in de lijst zitten, voor meer overzicht: https://github.com/Clarice-COD/the-client-website/blob/ccbb1475d044445e3fa4385b0fa73c3ce8e6343b/index.html#L14-L26