Closed Tolga1999 closed 8 months ago
vercel[bot]
commented
9 months ago The latest updates on your projects. Learn more about Vercel for Git ↗︎
| Name | Status | Preview | Comments | Updated (UTC) |
|---|---|---|---|---|
| toolgankelijk-sprint-18 | ✅ Ready (Inspect) | Visit Preview | 💬 Add feedback | Jan 21, 2024 10:36pm |
WesleySchorel
commented
9 months ago Het toevoegen lukt niet met een hoofdletter. Dit is verwarrend voor nieuwe gebruikers, misschien kan je een hint toevoegen of forceren dat de naam van een partner met kleine letters wordt geschreven. Het toevoegen met zonder hoofdletter werkt zoals afgesproken in de ticket: https://github.com/WesleySchorel/Toolgankelijk-Sprint-18/issues/26
Uit de lighthouse test kwamen de scores, 74 Performance en 94 Accessibility. We moeten dus nog even kijken naar de performance van deze pagina. Getest in Edge & Chrome.
De handmatige test leverde de gewenste bevindingen, de website werkt volledig met het toetsenbord en een screenreader leest alles op de juiste volgorde.
tom-2810
commented
9 months ago Zodra ik erop klik navigeer ik naar de /add route. Deze zou specifieker mogen, aangezien later ook een url toevoegen mogelijk moet zijn. Of we kunnen alles reactive bouwen, zodat de formData ook mee veranderd.
Een beetje los van deze pull request (POST), zou de tip linksboven vervangen kunnen worden met bijvoorbeeld een lijstje waarin alle bestaande partners onder elkaar staan. Simpel overzichtje.
Verder zijn de labels wel duidelijk. Deze worden ook voorgelezen door een screenreader, dus semantisch in orde.
Ik zie dat je client-side validation heb gebruikt: https://github.com/WesleySchorel/Toolgankelijk-Sprint-18/blob/d9c5d44d6c45eca7582f5d461aeb67e02caa1713/src/routes/add/%2Bpage.svelte#L26
Toch vangt je error dit goed af. Ik kan namelijk niet een partner toevoegen zonder url, zelfs wanneer ik deze in de dev-tools inspect weghaal. 👍
Maar als er iets mis is gegaan: error. Dan moet ik wel weer alles opnieuw invullen. Misschien is dit juist goed, maar je zou nog eens kunnen kijken naar een Snapshot. Daarmee voorkom je dat iemand een veld opnieuw moet invullen, wat een verbetering op UX gebied kan opleveren.
Is misschien een beetje flauw, maar ook na een paar seconden gebeurt er niets... ☝️🤓
En misschien is het leuk om de zojuist toegevoegde partner in de alert te plaatsen, zoals "GVB partner is succesvol toegevoegd. U wordt nu doorverwezen naar de partneroverzicht pagina!"
Maar het werkt gewoon!!! POSTENNNNNN
Tolga1999
commented
8 months ago Dankjewel voor alle feedback: PE toegepast en low capital input field zonder errors, dit allemaal is toegepast en werkend gesteld! Er zijn hier en daar nog kleine tweaks 😉
WesleySchorel
commented
8 months ago Feedback recap:
Alle verbetering toegepast zoals afgesproken. De lighthouse test geeft een score van 100 performance en een score van 95 toegankelijkheid.
Voor de rest niks op te merken. Goed bezig!
Het toevoegen van nieuwe partners met de hulp van een formulier, de user krijgt feedback als het formulier succesvol gesubmit is of als het formulier fouten heeft. De data wordt zonder enige problemen naar Hygraph gepost.