Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Componente Input: indagare ulteriormente la variante Password navigando da tastiera o con screen reader #1216

Closed
1 task done
Fupete opened this issue Sep 10, 2024 · 3 comments · Fixed by #1232
Closed
1 task done
Assignees
Labels
a11y Issue e PR riguardanti accessibilità e dintorni bug Issue e PR su problemi di funzionamento della libreria

Comments

@Fupete
Copy link
Contributor

Fupete commented Sep 10, 2024

Esiste già una discussione sul tema che ti interessa, o su un tema simile?

  • Ho verificato e non esiste

Versione della libreria

2.9.0

Cosa

La variante Password non permette di attivare il "Mostra password" navigando da tastiera o ascoltando da lettore di schermo.

Entrambe funzionanità che credo sarebbero necessarie e molto utili. Da verificare anche rispetto alle WCAG 2.2.

Update: C'è anche il tema che il "Mostra password" deve avere la semantica corretta (es. di un pulsante) e le alternative testuali adeguate (vedi commento di rscano che segue per dettagli).

Perché

Navigabile. Amichevole con i lettori di schermo. Comprensibile.

Contesto

Verifiche di accessibilità sui componenti.

⚠️ Sarà da aggiornare lo status delle verifiche nella board census alla risoluzione.

Altro

Per chi gestirà la risoluzione della presente segnalazione segnalo la documentazione del componente su Gov UK che può essere utile per raffronto: https://design-system.service.gov.uk/components/password-input/

Il problema è in caso presente anche su React e Angular kit, da gestire anche lì alla risoluzione della presente.

@Fupete Fupete added the a11y Issue e PR riguardanti accessibilità e dintorni label Sep 10, 2024
@Fupete Fupete changed the title Componente Input: variante Password navigando da tastiera o con screen reader Componente Input: indagare ulteriormente la variante Password navigando da tastiera o con screen reader Sep 10, 2024
@Fupete
Copy link
Contributor Author

Fupete commented Sep 10, 2024

Un contributo di @cfabry su questo sarebbe molto utile.

@rscano
Copy link

rscano commented Sep 10, 2024

Attualmente l'oggetto mostra password non è conforme alla EN / WCAG 2, in particolare al criterio di successo 2.1.1. Va quindi innanzitutto dato un ruolo adeguato (es: pulsante) fornendo anche alternative testuali adeguate (mostra password / nascondi password) in quanto va garantita anche la possibilità agli utenti non vedenti di gestire / farsi leggere la password (che altrimenti viene letta come punto elenco o similare).

Consiglio per approfondimenti: Password Forms — 3 Ways to Make Them More Accessible

@Fupete Fupete added the bug Issue e PR su problemi di funzionamento della libreria label Sep 10, 2024
@zetareticoli zetareticoli moved this from 🆕 New to 📋 Backlog in Design system .italia 🇮‍🇹 Sep 23, 2024
@Fupete Fupete moved this from 📋 Backlog to 🏗 In progress in Design system .italia 🇮‍🇹 Sep 30, 2024
@Fupete Fupete linked a pull request Sep 30, 2024 that will close this issue
4 tasks
@Fupete

This comment was marked as outdated.

@Fupete Fupete moved this from 🏗 In progress to 👀 In review in Design system .italia 🇮‍🇹 Oct 1, 2024
@Fupete Fupete moved this from 👀 In review to 🏗 In progress in Design system .italia 🇮‍🇹 Oct 1, 2024
@Fupete Fupete added this to the Prossimi Fix Accessibilità milestone Oct 1, 2024
@Fupete Fupete moved this from 🏗 In progress to 👀 In review in Design system .italia 🇮‍🇹 Oct 3, 2024
@github-project-automation github-project-automation bot moved this from 👀 In review to ✅ Done in Design system .italia 🇮‍🇹 Oct 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a11y Issue e PR riguardanti accessibilità e dintorni bug Issue e PR su problemi di funzionamento della libreria
Projects
Status: ✅ Done
Development

Successfully merging a pull request may close this issue.

2 participants