italia / design-comuni-pagine-statiche

Le pagine statiche che compongono il modello di sito web per i comuni
https://italia.github.io/design-comuni-pagine-statiche
BSD 3-Clause "New" or "Revised" License
18 stars 24 forks source link

Fix/arguments chips accessibility #67

Closed lorycade closed 1 year ago

vercel[bot] commented 1 year ago

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

Name Status Preview Updated
design-comuni-pagine-statiche ✅ Ready (Inspect) Visit Preview Nov 18, 2022 at 3:33PM (UTC)
lorycade commented 1 year ago

Ciao @lorycade, ho avuto modo di vedere la preview e ti segnalo di seguito le cose che non vanno bene:

  • in hover viene mostrato l'underline
  • il <div class="cmp-tag è superfluo, la classe non ha proprietà quindi si può rimuovere
  • ATTENZIONE 🚨tutte le chips ora riportano l'attributo data-element="service-topic" in ogni pagina, mentre credo sia necessario solo per la scheda servizio > controllare criterio del crawler
  • c'è uno spazio di troppo applicato a chip-label che fa disallineare il testo, nella versione precedente non c'era
  • i commit hanno modificato intestazione di molte righe di codice, è normale?
  • la classe .argomenti applicata al <div> contenitore non sembra fare nulla, si può rimuovere (solo per la lista in Homepage)
  • possiamo rimuovere tutti gli stili custom applicati ai chips (spazio, dimensione, line-height) che fanno override alle proprietà di Bootstrap Italia, è un occasione per ripulire > teniamo solo le personalizzazione di colore

@zetareticoli fatto tutto, per quanto riguardo le molte righe di codice, è dovuto al fatto che ho adeguato su tutte le pagine il componente (in alcune pagine era presente il codice direttamente in pagina). Se rilevi altre anomalie fammi sapere, grazie.