WesleySchorel / Toolgankelijk-Sprint-18

De Toolgankelijkheid website voor Vervoerregio Amsterdam om partners te controleren op toegankelijkheid.
https://toolgankelijk.vercel.app/
MIT License
3 stars 2 forks source link

Checklist posten #43

Closed tom-2810 closed 8 months ago

tom-2810 commented 8 months ago

Het posten van een project checklist naar Hygraph. Er word formData gepost naar het Hygraph CMS. Het formulier is niet afhankelijk van JavaScript. Wanneer dit wel aanwezig is, hoeft de pagina niet opnieuw geladen te worden, wat een soepele UX mogelijk maakt.

De post queries worden per niveau en per principe gestuurd, waardoor Hygraph niet onnodig wordt belast.

Nadat er op opslaan (rechts beneden) geklikt is, verschijnt er een toast/notificatie die feedback geeft dat de checklist succesvol is opgeslagen. Deze werkt ook vanuit de server.

vercel[bot] commented 8 months ago

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

Name Status Preview Updated (UTC)
toolgankelijk-sprint-18 ✅ Ready (Inspect) Visit Preview Jan 23, 2024 0:25am
DaanCarst commented 8 months ago

Ziet er denk ik prima uit!

tom-2810 commented 8 months ago

En wat vinden jullie van deze?

image

DaanCarst commented 8 months ago

goeie feedback voor bij het opslaan, misschien kan het nog iets groter?

WesleySchorel commented 8 months ago

goeie feedback voor bij het opslaan, misschien kan het nog iets groter?

Dit is een goede optie voor de volgende versie na de release canidate!

Overig: De website werkt volledig met het toetsenbord, je kan door alle items heen tabben.

De lighthouse test geeft gewenste resultaten. De performance is top met een score van 100. De accessibility geeft een score van 93. Op deze vlakken voldoet deze pull request aan de performance en toegankelijkheids eisen. image

tom-2810 commented 8 months ago

Er is een refresh nodig na het opslaan, als er geen JS werkt.

https://github.com/WesleySchorel/Toolgankelijk-Sprint-18/assets/112861614/22c8ffb9-5d1f-44e3-ad9d-0f9ad55fc6ce

WesleySchorel commented 8 months ago

Er is een refresh nodig na het opslaan, als er geen JS werkt.

opname_refresh.mp4

De commit voldoet nu aan genoeg eisen om gemerged te kunnen worden. Verdere enchancements zouden in de toekomst nog toegepast kunnen worden wanneer deze gewenst zijn.