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

fix: add a11y features to input password #1232

Conversation

Fupete
Copy link
Contributor

@Fupete Fupete commented Sep 30, 2024

fix: #1216

Obiettivi:

  • rendere accessibile il pulsante per visualizzare la password inserita;
  • verificare utilità e rischi, e opportunità di miglioramento e accessibilità, della variante con strength-meter;
    • verificare opportunità di aggiunta versione con suggerimenti rispetto ai requisiti, e verifiche su questi
  • verificare documentazione.

Copy link

vercel bot commented Sep 30, 2024

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

Name Status Preview Updated (UTC)
bootstrap-italia ✅ Ready (Inspect) Visit Preview Oct 8, 2024 2:38pm

@Fupete Fupete force-pushed the 1216-componente-input-indagare-ulteriormente-la-variante-password-navigando-da-tastiera-o-con-screen-reader branch from b4ec891 to bbbe4b4 Compare October 1, 2024 08:35
…-la-variante-password-navigando-da-tastiera-o-con-screen-reader
@Fupete Fupete self-assigned this Oct 1, 2024
@Fupete Fupete added a11y Issue e PR riguardanti accessibilità e dintorni enhancement bug Issue e PR su problemi di funzionamento della libreria labels Oct 1, 2024
…-la-variante-password-navigando-da-tastiera-o-con-screen-reader
@Fupete
Copy link
Contributor Author

Fupete commented Oct 8, 2024

@astagi sì, credo sia utile coinvolgere un esperto di sicurezza e linee guida NIST prima di procedere magari, più che altro per capire se l'indicazione di sicurezza abbia ancora senso oggi, o possa essere fuorviante così incorporata nel js. Oppure se serva magari qualche callout in documentazione per ricordare che la sicurezza...

Sennò potremmo sistemare intanto la UI con questa PR, direzione accessiblità, e poi aprire subito dopo una issue Urgente dedicata alla verifica dello score calculator, parliamone next week all'hands-on.

immagine

Aggiunto disclaimer callout sicurezza.

@Fupete
Copy link
Contributor Author

Fupete commented Oct 8, 2024

@astagi ti segnalo le ultime due cose che vedo:

  • nel campo input usato nel primo esempio che è nella pagina Introduzione ai form, a più colonne, essendo dentro una ´row´ salta qualcosa sul posizionamento assoluto del pulsante per mostrare/nascondere a destra, va fuori del campo (vedi img)
    immagine
* **( ✅ Risolto)** nel caso di atterraggio su una pagina dove l'autocompositore del browser ha già inserito la password nel campo, non si sposta la label, almeno su Firefox (vedi img)
  ![immagine](https://private-user-images.githubusercontent.com/6736103/373616133-cdd67a34-26a3-4b48-9acc-8022ecc971c2.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjgzOTgxNjcsIm5iZiI6MTcyODM5Nzg2NywicGF0aCI6Ii82NzM2MTAzLzM3MzYxNjEzMy1jZGQ2N2EzNC0yNmEzLTRiNDgtOWFjYy04MDIyZWNjOTcxYzIucG5nP1gtQW16LUFsZ29yaXRobT1BV1M0LUhNQUMtU0hBMjU2JlgtQW16LUNyZWRlbnRpYWw9QUtJQVZDT0RZTFNBNTNQUUs0WkElMkYyMDI0MTAwOCUyRnVzLWVhc3QtMSUyRnMzJTJGYXdzNF9yZXF1ZXN0JlgtQW16LURhdGU9MjAyNDEwMDhUMTQzMTA3WiZYLUFtei1FeHBpcmVzPTMwMCZYLUFtei1TaWduYXR1cmU9ZjI4MjgyMDBmOTdmZDlmMWU2ZmNkZDViNzZjZDNmNDk0ZTc4ODgyZDg1MTliY2RhNGFiNzYyOGQ5NjA4NjIxMSZYLUFtei1TaWduZWRIZWFkZXJzPWhvc3QifQ.N4EqcxLouGCocRt21DqnPVImoQBcuF5lsJ9nNZW0Lu8)

Risolto anche il check per l'autoFill che sposta automaticamente l'etichetta.
Rimane solo il problema eventuale delle due colonne qui mostrato.

docs/form/input.md Outdated Show resolved Hide resolved
@astagi astagi merged commit c2a7b22 into main Oct 11, 2024
6 of 7 checks passed
@astagi astagi deleted the 1216-componente-input-indagare-ulteriormente-la-variante-password-navigando-da-tastiera-o-con-screen-reader branch October 11, 2024 08:14
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 enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Componente Input: indagare ulteriormente la variante Password navigando da tastiera o con screen reader
4 participants