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 Sections: aria-describedby vs aria-labelledby #1199

Closed
1 task done
Mattia541993 opened this issue Aug 29, 2024 · 7 comments · Fixed by #1204
Closed
1 task done

Componente Sections: aria-describedby vs aria-labelledby #1199

Mattia541993 opened this issue Aug 29, 2024 · 7 comments · Fixed by #1204
Assignees
Labels
a11y Issue e PR riguardanti accessibilità e dintorni

Comments

@Mattia541993
Copy link
Contributor

Mattia541993 commented Aug 29, 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

Nella documentazione delle "Section" viene utilizzato l'attributo aria-describedby per creare una correlazione tra la <section> e l'intestazione <h2>. Da linee guida ARIA (Headings e Labeling Regions) sarebbe più corretto utilizzare l'attributo aria-labelledby.

Perché

Per la semantica della struttura della pagina

Contesto

Aggiornato da @Fupete 2/9/2024:

Verifiche di accessibilità sui componenti.

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

Altro

Aggiornato da @Fupete 3/9/2024:

Da "sistemare" anche nel React kit al completamento.

@Fupete Fupete added the a11y Issue e PR riguardanti accessibilità e dintorni label Aug 29, 2024
@Fupete Fupete self-assigned this Aug 29, 2024
@Fupete
Copy link
Contributor

Fupete commented Aug 29, 2024

Ciao @Mattia541993, grazie per questa e le altre segnalazioni di oggi.

Appena possibile faccio un giro a prioritizzarle e, nel tempo, le verrai pian piano assegnare per eventuali soluzioni o commenti.

Posso chiederti se dove la soluzione è evidente, come in questo caso, saresti anche in grado di aprire PR di soluzione? Per curiosità, e perché sarebbe molto utile a velocizzare.

Un caro saluto, Daniele T.

cc @astagi

@Fupete
Copy link
Contributor

Fupete commented Aug 29, 2024

Per informazione: nella board di censimento status ho aggiornato lo status del campo "Amichevole con i lettori di schermo" in "IN REVIEW", e aggiunto questa issue al tracciamento.

@Fupete Fupete removed their assignment Aug 29, 2024
@Mattia541993
Copy link
Contributor Author

Ciao! Grazie a te per la puntualità della risposta. Certamente posso aprire delle PR. Ci sono delle linee guida tecniche su come farle, tipo convenzioni di naming o branch dal quale aprirle?

Grazie mille!

@Fupete
Copy link
Contributor

Fupete commented Aug 30, 2024

Bene! Lascio ad @astagi le convenzioni 😄

@astagi
Copy link
Member

astagi commented Aug 30, 2024

Ciao! Grazie a te per la puntualità della risposta. Certamente posso aprire delle PR. Ci sono delle linee guida tecniche su come farle, tipo convenzioni di naming o branch dal quale aprirle?

Grazie mille!

Ciao @Mattia541993 e grazie per il supporto :) noi utilizziamo la convenzione del nome dei branch (tipo feat/[descrizione_problema][numero_eventuale_di_issue] o fix/[descrizione_problema][numero_eventuale_di_issue] nel caso di fix) non è comunque fondamentale, invece obbligatoria è l'utilizzo di commit semantici.

@Mattia541993
Copy link
Contributor Author

Ciao, grazie a te per la risposta. Mi scuso per aver fatto la domanda sulle convenzioni quando in realtà è tutto ben documentato.

@Fupete Fupete moved this from 🆕 New to 🏗 In progress in Design system .italia 🇮‍🇹 Sep 3, 2024
@Fupete Fupete linked a pull request Sep 3, 2024 that will close this issue
5 tasks
@Fupete Fupete moved this from 🏗 In progress to 👀 In review in Design system .italia 🇮‍🇹 Sep 3, 2024
@github-project-automation github-project-automation bot moved this from 👀 In review to ✅ Done in Design system .italia 🇮‍🇹 Sep 16, 2024
@Fupete
Copy link
Contributor

Fupete commented Sep 17, 2024

@astagi la soluzione qui implementata va riportata anche in React kit. Apri tu una issue?

@Fupete Fupete reopened this Sep 17, 2024
@github-project-automation github-project-automation bot moved this from ✅ Done to 📋 Backlog in Design system .italia 🇮‍🇹 Sep 17, 2024
@Fupete Fupete closed this as completed Nov 11, 2024
@github-project-automation github-project-automation bot moved this from 📋 Backlog to ✅ Done in Design system .italia 🇮‍🇹 Nov 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
Projects
Status: ✅ Done
3 participants