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 hero: review a11y #1092

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

Componente hero: review a11y #1092

Fupete opened this issue Aug 23, 2024 · 1 comment · May be fixed by #1104
Labels

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

Le varianti non sembrano seguire la struttura del componente in Bootstrap Italia. Probabilmente la presente struttura deriva da una precedente versione di Bootstrap Italia e c'è stata divergenza.

In particolare:

  • Il componente dovrebbe essere racchiuso dentro una section, e laddove necessario (varianti immagine) conaria-label dedicata.
  • Da verificare inoltre l'uso di h2al posto di h1 negli esempi per miglior interpretazione della corretta gerarchia dei titoli.
  • C'è un bug nell'ultima variante con testo sovrapposto, all'immagine manca l'overlay di colore grigio scuro per garantire l'accessibilità del testo.

Perché

Coerenza tra le risorse di Design system .italia. Amichevole con i lettori di schermo. Visivamente accessibile.

Contesto

Verifiche di accessibilità sui componenti.

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

Altro

Si segnala la issue aperta anche su Bootstrap Italia italia/bootstrap-italia#1175, specialmente in merito al deprecare la variante con testo sovrapposto e al ripensare lo span Categoria nelle varianti con testo. Da sincronizzare le soluzioni.

@Virtute90 Virtute90 linked a pull request Sep 12, 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
Projects
Status: 🆕 New
Development

Successfully merging a pull request may close this issue.

2 participants