search

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

Errore padding componente Accordion #116

Closed AlessandroVecchi closed 10 months ago

AlessandroVecchi commented 1 year ago

Ciao, https://github.com/italia/design-comuni-pagine-statiche/blob/main/src/components/cmp-accordion/_cmp-accordion.scss Alla riga 101

      @include media-breakpoint-up(sm) {
        padding-left: 0.875rem 0 0;
      }
stale[bot] commented 11 months ago

Questo problema è stato automaticamente contrassegnato come non aggiornato perché non ha avuto attività recentemente. Sarà chiuso se non si verificano ulteriori attività. Considerate inoltre l'aggiornamento alle ultime versioni del tema. Grazie.

enrimk commented 8 months ago

Il bug è ancora lì.

Per chiarire: la regola è sintatticamente invalida, padding-left prende un valore solo, non tre come invece può fare padding. Quindi la riga non ha effetto, e se non serve è meglio toglierla.

A cosa serve la regola? Si applica a .accordion-header .accordion-button. È un piccolo riallineamento su mobile (tablet) dei titoli dell'accordion in pagine come questa (tab Attività), ma l'allineamento non sarebbe comunque preciso, e non si riesce a ricavare dal contesto se si intendeva usare padding o padding-left.