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

docs: changed aria-describedby to aria-labelledby in the Section component #1204

Merged
merged 1 commit into from
Sep 16, 2024

Conversation

Mattia541993
Copy link
Contributor

Cambio in aria-labelledby da aria-describedby nel componente Section

Descrizione

In riferimento alla issue 1199 ho modificato i vari aria-describedby presenti nella pagina della documentazione del componente Section in aria-labelledby in quanto più aderenti alle linee guida ARIA

Checklist

Copy link

vercel bot commented Sep 3, 2024

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

Name Status Preview Updated (UTC)
bootstrap-italia ✅ Ready (Inspect) Visit Preview Sep 3, 2024 7:53am

@Mattia541993
Copy link
Contributor Author

Ho visto all'interno del codice del repository nella cartella api/componenti che è presente un file sections.json nel quale non ho risolto i problemi di aria-describedby perchè non ho capito come quella parte viene usata all'interno del codice. Data la cartella api ho pensato potesse essere una convenzione della vostra piattaforma di deploy per esporre appunto delle API, però prima di proporre delle modifiche volevo chiedervi appunto dei chiarimenti.

Grazie mille

@Fupete
Copy link
Contributor

Fupete commented Sep 3, 2024

@Mattia541993 i file di API di Bootstrap Italia sono generati automaticamente dagli esempi in documentazione, ci pensa un'azione GitHub dedicata. Non c'è bisogno di editare quel file quindi. È la API usata per mostrare le varianti nella documentazione sul sito Designers Italia.

Credo che se togliamo dalla PR le modifiche al file gemlock si possa mergiare @astagi, ti metto reviewer per dare un'occhiata. Le modifiche saranno da riportare anche nel React kit, facendole contestualmente, aggiornando e tenendo aperta la issue collegata per tracciarlo, o aprendone una dedicata sul suo repo.

@Fupete Fupete linked an issue Sep 3, 2024 that may be closed by this pull request
1 task
@Fupete Fupete requested a review from astagi September 3, 2024 09:14
@Mattia541993
Copy link
Contributor Author

Le modifiche al Gemfile.lock le ho guardate perchè effettivamente sono sembrate strane anche a me, ma guardandole in dettaglio non rendono meno dipendenti dalla piattaforma le varie librerie?

@astagi astagi marked this pull request as ready for review September 10, 2024 13:23
@astagi
Copy link
Member

astagi commented Sep 10, 2024

@Mattia541993 grazie per la PR, si in effetti sono dipendenze strettamente legate alla piattaforma. Forse un refuso? Hai utilizzato Docker per avviare il progetto oppure con ambiente Ruby sulla tua macchina?

@astagi astagi merged commit a4ece2d into italia:main Sep 16, 2024
7 checks passed
@astagi
Copy link
Member

astagi commented Sep 16, 2024

@Mattia541993 il merge è andato a buon fine e anche la parte di Ruby che gira su Docker, lasciamo il lock come in questa PR :) grazie per il contributo!

@Mattia541993
Copy link
Contributor Author

Scusami se non ti ho più risposto. ero in ferie... Sì, ho usato docker. Grazie mille per l'approvazione del merge!!

@astagi
Copy link
Member

astagi commented Sep 17, 2024

Figurati @Mattia541993 grazie a te :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Componente Sections: aria-describedby vs aria-labelledby
3 participants