delta10 / signalen-frontend-wcag

https://signalen-frontend-wcag.vercel.app
European Union Public License 1.2
0 stars 1 forks source link

Voeg melding toe voor wanneer het submitten faalt #10 #52

Open iehkaatee opened 3 days ago

iehkaatee commented 3 days ago

Vraag voor @TessaViergever of @Robbert; wanneer de gebruiker bijlages heeft toegevoegd willen we wachten met navigeren naar de succes pagina tot deze succesvol zijn geupload. Wat denken jullie dat de beste manier is om het laden weer te geven op de summary pagina? En hoe zorgen we ervoor dat het ook duidelijk is voor gebruikers die navigeren op basis van een screen reader?

vercel[bot] commented 3 days ago

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

Name Status Preview Comments Updated (UTC)
signalen-frontend-wcag ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 1, 2024 9:45am
Robbert commented 3 days ago

@iehkaatee

iehkaatee commented 3 days ago

@iehkaatee

  • We kunnen de submit button disabled maken met aria-disabled="true" zolang de upload bezig is. Submitten blijft wel mogelijk, maar het is een duidelijke hint.
  • We kunnen een melding voor de button group plaatsen met "Je kunt door naar volgende stap wanneer het uploaden van bestanden klaar is." De melding kan met aria-describedby gekoppeld worden aan de submit button.
  • Wanneer je wel submit terwijl het uploaden niet klaar is, dan kunnen we een foutmelding toevoegen aan de Alert. "Wacht tot het uploaden van {label} klaar is."
  • We kunnen een aria-live="assertive" positieve Alert tonen bij het upload-veld wanneer alle uploads klaar zijn. Dan weet de screenreader gebruiker die nog focus heeft op de button ook dat het zover is.

Ah super, hier kan ik wat mee! bedankt!

iehkaatee commented 3 days ago

@Robbert Zou je nog een keer willen kijken naar hoe ik het nu heb geimplementeerd? Ben benieuwd of dit goed is zo.

Daarnaast heb ik nog een vraag, wanneer ik met een screenreader het formulier probeer te submitten wordt de tekst de eerste keer helemaal voorgelezen zoals ik verwacht. Maar zodra ik het formulier na een error opnieuw probeer te submitten wordt de tekst in het loading alert niet nog een keer voorgelezen. Heb jij een idee waarom dit niet werkt?

iehkaatee commented 1 day ago

Ik denk wat er nog mist is dat als er attachments zijn, dat er een setComplete(true) of iets dergelijks is die een complete && <Alert role="status">Het uploaden is klaar.</Alert> triggert.

Omdat je automatisch naar de volgende pagina gaat lijkt het me niet handig om hier een Alert met "verwerken is succesvol afgerond" te plaatsen. Dit wilde ik met een ander issue later oppakken