search

Sascha-davidson / lets-jam-webapplicatie

Ontwerp en ontwikkel een complexe, gebruiksvriendelijke en interactieve webapplicatie op basis van een bestaande api of een zelf in te richten headless CMS
https://lets-jam-webapplicatie-xi.vercel.app/
MIT License
1 stars 4 forks source link

Style the 'Player Lists' in Statreader page #40

Open Siem-P opened 9 months ago

Siem-P commented 9 months ago

Design uit Figma

**Wat moet er nog gebeuren om het af te maken** - [ ] Is het component responsive en voldoet het aan de huisstijl? - [ ] Voor responsive moet er nog wat gebeuren. Nog niet op alle apparaten getest - [ ] Kan nog een breakpoint worden toegevoegd voor M-scherm - [ ] Small screen doet het **Voldoet het component aan de coding standards?** - [x] Gebruik gemaakt van Sveltekit - [x] HTML semantisch geschreven - [x] CSS classes gebruikt gemaakt van kebab-casing & id's van Camelcasing - [x] Javascript variabelen imports, definiëren, eventListeners en functies gegroepeerd - [ ] Prettier css ordering - [x] Tabbing spacing op 4 spaces - [ ] Consistent gebruik van semi-colons of juist geen - [ ] Comments bij code: * Functies: beschrijf in een regel wat de functie doet. * Components : Beschijf in de html per component wat het inhoud, kort & bondig. **Is het component toegankelijk?** - [ ] Focus test gedaan - [ ] Nog geen handmatige tests gedaan - [x] Light house **Is het component performant?** - [ ] Geen layout shift of andere fratsen te zien? - [ ] Alle afbeeldingen moeten nog worden toegevoegd en geoptimaliseerd - [ ] Image sizes staan in de HTML in de template - [ ] Loading="lazy" staat in de HTML - [ ] Afbeeldingen hebben een vaste width & height
Tristandemuijnck commented 9 months ago

User test - Tristan

Ik heb een user test gedaan op het player list component. Ik heb mij gehouden aan de bovenstaande checklists om de user test te voltooien.

Responsive

Op het gebied van responsiveness kan er nog wat gebeuren op het gebied van tablet size. Ik heb begrepen dat de applicatie vooral voor tablet en groter ontworpen is, maar de kleinste tablet size (768px width) komt niet helemaal lekker uit.

Verder heb je een probleem als een naam te lang wordt. De naam gaat buiten beeld en zorgt ervoor dat er een horizontal scroll onstaat.