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 list: usare corretta semantica se c'è testo in più... #1223

Closed
1 task done
Fupete opened this issue Sep 17, 2024 · 0 comments · Fixed by #1252
Closed
1 task done

Componente list: usare corretta semantica se c'è testo in più... #1223

Fupete opened this issue Sep 17, 2024 · 0 comments · Fixed by #1252
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 17, 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 varianti con "testo aggiuntivo" e "con metadata", e le varianti per "menu multilinea...", abbisognano di un corretto approccio con heading h e p. E non dei soli due span oggi presenti.

La discussione è già stata affrontata in passato con @cfabry in questa issue: #764, e sarebbe stata in parte risolta almeno per la variante con Testo aggiuntivo da questa PR: #816... purtroppo sono state entrambe chiuse da Stale (oggi non più presente) prima di finire i lavori e rilasciarli.

Va, secondo me, recuperata quella PR e verificato se ci sono cose ancora migliorabili a livello visivo. Oltre ad aggiungere la soluzione per la variante "con metadata", "menu multilinea...".
Diventa ovviamente una breaking-change su quella variante del componente lista.

Perché

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

Sarà da riportare anche in React kit e Angular kit alla risoluzione.

Sarebbe ottimo verificare e risolvere anche in UI Kit Italia adottando gli stili di testo corretti di heading e paragrafi nella variante con testo aggiuntivo. cc @zetareticoli

Il componente è usato estensivamente anche nel sito Designers Italia per le liste di download risorse, vedi questa issue: italia/designers.italia.it#829

@Fupete Fupete added bug Issue e PR su problemi di funzionamento della libreria a11y Issue e PR riguardanti accessibilità e dintorni labels Sep 17, 2024
@Fupete Fupete changed the title Componente list: usare corretta semantica se c'è "testo aggiuntivo" Componente list: usare corretta semantica se c'è testo in più... Sep 17, 2024
@zetareticoli zetareticoli moved this from 🆕 New to 📋 Backlog in Design system .italia 🇮‍🇹 Sep 20, 2024
@Fupete Fupete added this to the Prossimi Fix Accessibilità milestone Oct 7, 2024
@zetareticoli zetareticoli moved this from 📋 Backlog to 🏗 In progress in Design system .italia 🇮‍🇹 Oct 16, 2024
@zetareticoli zetareticoli linked a pull request Oct 16, 2024 that will close this issue
5 tasks
@zetareticoli zetareticoli moved this from 🏗 In progress to 👀 In review in Design system .italia 🇮‍🇹 Oct 16, 2024
@github-project-automation github-project-automation bot moved this from 👀 In review to ✅ Done in Design system .italia 🇮‍🇹 Nov 4, 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