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 Button: ambiguità per la presenza di nome accessibile "link" e ambiguità con i testi semplici. #1259

Closed
1 task done
cfabry opened this issue Oct 25, 2024 · 2 comments · Fixed by #1392
Closed
1 task done
Assignees
Labels
a11y Issue e PR riguardanti accessibilità e dintorni

Comments

@cfabry
Copy link
Collaborator

cfabry commented Oct 25, 2024

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

  • Ho verificato e non esiste

Versione della libreria

2.10

Cosa

https://italia.github.io/bootstrap-italia/docs/componenti/buttons/

  1. Nel primo esempio, ma poi anche negli altri e anche su Designers Italia, abbiamo l'elemento <button> con nome accessibile "link". Questo crea confusione tra i due componenti.
  2. Negli esempi base (anche su Designers) per la classe .btn abbiamo un elemento senza alcuna distinzione visiva con un testo semplice.

Perché

  1. Il valore semantico di link e button è completamente diverso quindi opportuno eliminare ovunque la parola "link".
  2. Immagino sia necessario per avere un elemento molto lineare da inserire in altri contenitori, esempio in liste di pulsanti già separati da elementi <li>, tuttavia può risultare ambiguo e indurre gli sviluppatori a creare elementi <button> indistinguibili dai testi normali (errore di accessibilità molto frequente).
    Valutare se inserire un alert per questo problema, o, magari, valutare se possibile una soluzione alternativa che non permetta ai pulsanti di essere confusi con il testo.

Contesto

No response

Altro

No response

@cfabry cfabry added the a11y Issue e PR riguardanti accessibilità e dintorni label Oct 25, 2024
@Fupete Fupete moved this from 🆕 New to 📋 Backlog in Design system .italia 🇮‍🇹 Oct 28, 2024
@Fupete
Copy link
Contributor

Fupete commented Feb 10, 2025

@cfabry concordo che come minimo la variante "Link" dovrebbe avere un testo "Link azione" più chiaro (cose che succedono in pagina) e poi essere del colore del link (blu nel ns caso) e sottolineata già di base.

Anche io ho da sempre dubbi sull'utilità di questa variante che non sembra un pulsante... Però, oltre a componenti composti, in effetti può essere utile forse a quelle situazioni in cui l'azione cambia cose in pagina ma non deve necessariamente sembrare un pulsante. Penso a un "Leggi di più" che apre un collapse di testo ulteriore per esempio a seguire...

Sarebbe utile forse sì anche ripensare il contenuto del callout accessibilità che già c'è in documentazione se non è abbastanza chiaro. Ora è così:

Image

Metto in CC @zetareticoli che sta facendo il lavoro sui Design Tokens dove si potrebbe sistemare i colori intanto.

@zetareticoli zetareticoli moved this from 📋 Backlog to 🏗 In progress in Design system .italia 🇮‍🇹 Feb 11, 2025
@zetareticoli zetareticoli linked a pull request Feb 11, 2025 that will close this issue
5 tasks
@zetareticoli zetareticoli self-assigned this Feb 11, 2025
@zetareticoli
Copy link
Member

Ho aperto la PR #1392 con anteprima delle modifiche alla documentazione e al codice. In sostanza, la sola classe .btn non può essere utilizzata per conferire colore e aspetto di un pulsante. Deve per forza essere abbinata alle altre classi di tipo .btn-. Quelli di Bootstrap l'avevano pensata così.

Pertanto ho aggiornato il primo paragrafo di documentazione specificando meglio questo punto.

@Fupete Fupete moved this from 🏗 In progress to 🔖 Ready in Design system .italia 🇮‍🇹 Feb 12, 2025
@github-project-automation github-project-automation bot moved this from 🔖 Ready to ✅ Done in Design system .italia 🇮‍🇹 Feb 24, 2025
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
Projects
Status: ✅ Done
Development

Successfully merging a pull request may close this issue.

3 participants