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

fix: list link semantics and states #1252

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

Conversation

zetareticoli
Copy link
Member

Descrizione

La proprietà text-decoration: underline viene applicata di default a tutti i link che fanno parte di liste semplici.

Rimosso lo stato active perché si tratta di liste che non appartengono a menu di navigazione (c'è un codice specifico per quelle). Pertanto è superfluo.

Checklist

Copy link

vercel bot commented Oct 16, 2024

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

Name Status Preview Updated (UTC)
bootstrap-italia ✅ Ready (Inspect) Visit Preview Oct 24, 2024 3:45pm

@zetareticoli zetareticoli self-assigned this Oct 16, 2024
@zetareticoli zetareticoli linked an issue Oct 16, 2024 that may be closed by this pull request
@zetareticoli zetareticoli changed the title fix: link state in lists fix: link semantics and states Oct 16, 2024
@zetareticoli zetareticoli changed the title fix: link semantics and states fix: list link semantics and states Oct 16, 2024
@zetareticoli zetareticoli linked an issue Oct 16, 2024 that may be closed by this pull request
1 task
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.

Il tema gerarchia e semantica è presente anche per le varianti "con metadata", "menu multilinea...".

@zetareticoli
Copy link
Member Author

zetareticoli commented Oct 18, 2024

@Fupete tutte le indicazioni sullo stato "attivo" sono un po' fuorvianti, perché a meno che non si è in un contesto di menu primario o secondario, le liste di link non possono prevedere questo stato.

Sarei per rimuovere l'esempio active da tutte le liste, lasciandolo solo nei componenti dedicati o nelle varianti liste per menu.


Ho aperto la issue #1255 sul tema

@Fupete
Copy link
Contributor

Fupete commented Oct 18, 2024

@Fupete tutte le indicazioni sullo stato "attivo" sono un po' fuorvianti, perché a meno che non si è in un contesto di menu primario o secondario, le liste di link non possono prevedere questo stato.

Sarei per rimuovere l'esempio active da tutte le liste, lasciandolo solo nei componenti dedicati o nelle varianti liste per menu.

Lascio a te questa valutazione, a me torna tendenzialmente. Immagino ci siano eccezioni per le liste dropdown e per eventuali liste in back office che potrebbero averne bisogno. Ma sono eccezioni appunto.

Però il mio commento sopra era per altro: la soluzione

(o altra) va implementata anche per le varianti "con metadata" e "menu multilinea" intendevo, che hanno lo stesso problema di mancanza di struttura semantica tra titoletto e paragrafo in pratica. Andrà anche documentata probabilmente con qualcosa tipo un callout accessibilità che contenga una indicazione tipo "il livello di heading h degli esempi dovrà essere cambiato con il corretto livello a seconda della gerarchia nel contesto", o qualcosa del genere. Magari possiamo usare la stessa formula della PR per la nuova timeline.

@zetareticoli
Copy link
Member Author

@Fupete tutte le indicazioni sullo stato "attivo" sono un po' fuorvianti, perché a meno che non si è in un contesto di menu primario o secondario, le liste di link non possono prevedere questo stato.
Sarei per rimuovere l'esempio active da tutte le liste, lasciandolo solo nei componenti dedicati o nelle varianti liste per menu.

Lascio a te questa valutazione, a me torna tendenzialmente. Immagino ci siano eccezioni per le liste dropdown e per eventuali liste in back office che potrebbero averne bisogno. Ma sono eccezioni appunto.

Però il mio commento sopra era per altro: la soluzione

(o altra) va implementata anche per le varianti "con metadata" e "menu multilinea" intendevo, che hanno lo stesso problema di mancanza di struttura semantica tra titoletto e paragrafo in pratica. Andrà anche documentata probabilmente con qualcosa tipo un callout accessibilità che contenga una indicazione tipo "il livello di heading h degli esempi dovrà essere cambiato con il corretto livello a seconda della gerarchia nel contesto", o qualcosa del genere. Magari possiamo usare la stessa formula della PR per la nuova timeline.

Sì, ho aggiornato le varianti metadata e multilinea aggiungendo il tag <h4>

@Fupete
Copy link
Contributor

Fupete commented Oct 19, 2024

Sullo stato .active ti ho lasciato un commento nella issue dedicata: #1255 (comment)

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