OniWithTheHoodie commented 1 month ago

tabs menu gemaakt

tab-component

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

Punten om op te letten

-deze tab menu is bedoel om te navigeren door de pagina's

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] commented 1 month ago

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 10, 2024 9:25am
lose-your-head-the-client-case-5q6r	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Oct 10, 2024 9:25am

TomDeeterink1 commented 1 month ago

Ik heb alles getest en nagelopen voor dit element. Hij heeft een goeie score op lighthouse, alleen het contrast van de buttons moet besproken worden in de sprint review.

Ook heb ik 2 merge conflicts opgelost

OniWithTheHoodie commented 3 weeks ago