Pull Request Mobile nav#42

[TomDeeterink1]

Wat heb ik gemaakt? Ik heb het mobile menu gemaakt voor Mediahuis.

Punten om op te letten

• Kijk goed naar de files die er bij komen, en dat de layout file intact blijft.
• De icons moeten ook mee mergen.

---

Welke testen heb ik uitgevoerd? ik ben onze hele DOD afgegaan en heb de functionaliteit ook getest.

Code

• [x] Je code voldoet aan de afgesproken conventies
• [x] Je bent consistent met het gebruiken van tabs en enters
• [x] Je structureert je code met comments
• [x] Je legt ingewikkelde code uit met comments

Ontwerp

• [x] Het afgeronde component komt overeen met het ontwerp

• [x] Het is responsive op alle schermen tot minstens 300px breed

Toegankelijkheid

• [x] Is te bedienen met toetsenbord
• [x] Heeft een duidelijke focus state
• [x] Er wordt gebruik gemaakt van semantische html (probeer divs te voorkomen)
• [x] Op alle buttons en overige klikbare elementen staat cursor: pointer
• [x] Je scoort minstens een 85% op Lighthouse voor zowel mobiel als desktop op toegankelijkheid
• [x] Bij afbeeldingen, buttons en links staan alt tags die kort en duidelijk beschrijven wat het doet
• [x] Je html komt door de html validator
• [x] Er is op meerdere browsers getest, minstens firefox en chrome

Performance

• [x] Alle afbeeldingen hebben een standaard hoogte en een standaard breedte meegekregen in de html
• [x] Het formaat van elke afbeelding is webp of svg
• [x] Je hebt geen onnodige / dubbele code
• [x] Je behaalt een 85% op de performance Lighthouse test op zowel mobiel als desktop

Mergen

• [x] Alle merge conflicts zijn opgelost
• [x] Minstens 1 persoon heeft je code beoordeeld en goedgekeurd

Wat moet er getest worden? Ik zou graag willen dat er iemand extra controlleert of:

Alle functionaliteiten onder het kopje hoe werkt het, werken Of alles volgens code conventies is Of ik mij aan alle DOD's heb gehouden

[vercel[bot]]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
lose-your-head-the-client-case	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Oct 9, 2024 10:07am
lose-your-head-the-client-case-5q6r	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Oct 9, 2024 10:07am

[OniWithTheHoodie]

Ik heb het getest en het is Tabbaar en werkt ook met de screen reader

Mobiel heb je een score van 95

Desktop heb je een score van 93

Performance test ziet er goed uit goede resultaat

Code voldoet niet aan de conventies BEM verder heb je goeie indenting en niet te veel white spacing.