Codice css ridondante e non necessario nel componente toolbar #1747
Copy link
Copy link
Closed
Description
Esiste già una discussione sul tema che ti interessa, o su un tema simile?
- Ho verificato e non esiste
Versione della libreria
3.x
Cosa
Ora che ci sono i token, alcune regole presenti nel codice css di toolbar sono ridondanti.
Ad esempio:
- https://github.com/italia/bootstrap-italia/blob/3.x/src/scss/components/_toolbar.scss#L325
- https://github.com/italia/bootstrap-italia/blob/3.x/src/scss/components/_toolbar.scss#L328
- https://github.com/italia/bootstrap-italia/blob/3.x/src/scss/components/_toolbar.scss#L331
- https://github.com/italia/bootstrap-italia/blob/3.x/src/scss/components/_toolbar.scss#L335
- https://github.com/italia/bootstrap-italia/blob/3.x/src/scss/components/_toolbar.scss#L343
- https://github.com/italia/bootstrap-italia/blob/3.x/src/scss/components/_toolbar.scss#L346
Inoltre:
- si potrebbero introdurre anche nuovi token (tipo per il posizionamento e dimensione dei badge), in modo da ridurre il codice css generato.
- la regola
.it-icocredo non abbia piu senso (non l'ho mai vista usare negli esempi di toolbar) e si potrebbe togliere - anche la classe
.toolbar-alertnon l'ho mai vista usare negli esempi e forse si potrebbe eliminare come regola ( a meno che non sia lì per una questione di retrocompatibilità)
Perché
ottimizzazione e chiarezza del codice
Contesto
No response
Altro
No response
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
Type
Projects
- StatusShow more project fields✅ Done
Activity
refactor: remove unused icon styles from toolbar component #1747
zetareticoli commentedon Apr 22, 2026
@giuliaghisini posso chiederti perchè ritieni ridondanti le declarations evidenziate?
Ho rimosso dal codice CSS la classe
.it-icointanto.giuliaghisini commentedon Apr 22, 2026
@zetareticoli
https://github.com/italia/bootstrap-italia/blob/3.x/src/scss/components/_toolbar.scss#L325
qui ad esempio, queste regole
sono ridondanti e inutili, perchè hai già assegnato alla variabile css il valore che ti serve in quel caso. Quella regola per color e fill dell'icona è già definita qui https://github.com/italia/bootstrap-italia/blob/3.x/src/scss/components/_toolbar.scss#L288
fix: remove duplicated properties in toolbar #1747
zetareticoli commentedon Apr 23, 2026
Risolto in 9b760b3