italia / bootstrap-italia

Bootstrap Italia è un tema basato su Bootstrap 5 conforme alle "Linee guida di design per i siti internet e i servizi digitali della Pubblica Amministrazione"
https://italia.github.io/bootstrap-italia
BSD 3-Clause "New" or "Revised" License
308 stars 163 forks source link

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

Open Mattia541993 opened 2 weeks ago

Mattia541993 commented 2 weeks ago

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

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 commented 2 weeks ago

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 commented 2 weeks ago

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.

Mattia541993 commented 2 weeks ago

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 commented 2 weeks ago

Bene! Lascio ad @astagi le convenzioni 😄

astagi commented 2 weeks ago

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/[descrizioneproblema][numero_eventuale_di_issue] o fix/[descrizioneproblema][numero_eventuale_di_issue] nel caso di fix) non è comunque fondamentale, invece obbligatoria è l'utilizzo di commit semantici.

Mattia541993 commented 2 weeks ago

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

Fupete commented 20 hours ago

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