Merge Header component #12

[TomDeeterink1]

Wat heb ik gemaakt? Ik heb de header gemaakt voor Mediahuis, om verder te werken moet ik eerst mergen met de main.

Een volledige uitgebreide pull request komt einde deze week.

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.
• Het klopt dat de kleuren nog geen var zijn, dit doe ik gelijk na de merge.

---

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

[driezie]

De header ziet er goed uit, het klopt ook volgens het design en voldoet aan onze afgesproken conventies.