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

Rimozione dei riferimenti a "elemento disabilitato" nella Toolbar sugli elementi disabilitati #1251

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

Mattia541993
Copy link
Contributor

@Mattia541993 Mattia541993 commented Oct 15, 2024

Nella toolbar grande accanto all'elemento disabilitato "Preferiti" era presente uno <span class="visually-hidden">elemento disabilitato</span> che, non avendo uno spazio dal testo precedente "preferiti", veniva letto dagli screen reader come un'unica parola. Dato che nell'esempio l'elemento in questione è già inserito in un div aria-disabled="true" fornire questa ulteriore precisazione è ridondante, pertanto l'ho tolta dall'esempio e dalla parte descrittiva della documentazione che la menziona

Fixes #1192

Checklist

Copy link

vercel bot commented Oct 15, 2024

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

Name Status Preview Updated (UTC)
bootstrap-italia ✅ Ready (Inspect) Visit Preview Nov 4, 2024 5:28pm

@Mattia541993
Copy link
Contributor Author

Non capisco il perchè Pa11y salta.. il file "liste di immagini" non l'ho toccato

@Fupete Fupete self-requested a review October 18, 2024 07:50
Copy link
Contributor

@Fupete Fupete left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Mattia541993 toglierei dalla PR tutti questi spazi aggiunti senza motivo se possibile.

immagine

@astagi
Copy link
Member

astagi commented Oct 30, 2024

Non capisco il perchè Pa11y salta.. il file "liste di immagini" non l'ho toccato

tranquillo @Mattia541993 va semplicemente in timeout, proverò ad aumentare il tempo di esecuzione massimo

@astagi
Copy link
Member

astagi commented Oct 31, 2024

@Mattia541993 la seguente PR è pronta per la revisione? possiamo levarla dalla draft nel caso?

@Mattia541993
Copy link
Contributor Author

Mattia541993 commented Nov 4, 2024

@Mattia541993 la seguente PR è pronta per la revisione? possiamo levarla dalla draft nel caso?

Ciao, se capissi perchè la pipeline fallisce in Pa11y sì, potremmo metterla in ready. Ho anche tolto gli spazi inutili dal codice che mi aveva aggiunto il linter. Nel senso: è andata in timeout vedo, ma non saprei come aumentare il tempo di questo check

@astagi
Copy link
Member

astagi commented Nov 4, 2024

@Mattia541993 la seguente PR è pronta per la revisione? possiamo levarla dalla draft nel caso?

Ciao, se capissi perchè la pipeline fallisce in Pa11y sì, potremmo metterla in ready. Ho anche tolto gli spazi inutili dal codice che mi aveva aggiunto il linter. Nel senso: è andata in timeout vedo, ma non saprei come aumentare il tempo di questo check

è un maledetto timeout ogni tanto lo fa 👎 non dipende dal tuo codice tranquillo, metto in ready e chiedo review grazie @Mattia541993

@astagi astagi requested a review from Fupete November 4, 2024 15:35
@astagi astagi marked this pull request as ready for review November 4, 2024 15:36
@astagi
Copy link
Member

astagi commented Nov 6, 2024

@Mattia541993 ho segnato questa PR come risolutiva della issue #1192, ci sono altre issue che avevi aperto e che va a risolvere?

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 Toolbar: Criticità di accessibilità relative al layout
3 participants