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 modale: h2 per i titoli, non h5 e link sbagliato a doc mdn #1093

Open
1 task done
Fupete opened this issue Aug 23, 2024 · 1 comment · May be fixed by #1105
Open
1 task done

Componente modale: h2 per i titoli, non h5 e link sbagliato a doc mdn #1093

Fupete opened this issue Aug 23, 2024 · 1 comment · May be fixed by #1105
Labels
a11y Accessibility enhancement

Comments

@Fupete
Copy link

Fupete commented Aug 23, 2024

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

  • Ho verificato e non esiste

Versione della libreria

5.2.0

Cosa

Come in Bootstrap Italia e in Angular kit si consiglia di usare in tutte le varianti il tag h2 per il titolo della modale, al posto dell'attuale di h5. Per rispettare la corretta gerarchia degli heading di pagina.
Aggiungendo poi la classeh5 per la corretta dimensione del testo.

Si segnala inoltre che uno dei link MDN in apertura porta a una pagina 404.

Perché

Navigabile. Documentazione comprensibile.

Contesto

Verifiche di accessibilità sui componenti.

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

Altro

Ci sarà un'altra cosa da sincronizzare nella variante con icona, dopo la risoluzione di questa su Bootstrap Italia: italia/bootstrap-italia#1176

@Fupete Fupete added enhancement a11y Accessibility labels Aug 23, 2024
@Fupete Fupete changed the title Componente modale: h2 per i titoli, non h5 Componente modale: h2 per i titoli, non h5 e link sbagliato a doc mdn Aug 23, 2024
@Virtute90 Virtute90 linked a pull request Sep 13, 2024 that will close this issue
4 tasks
@Virtute90
Copy link
Contributor

Inserita PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a11y Accessibility enhancement
Projects
Status: 🆕 New
Development

Successfully merging a pull request may close this issue.

2 participants