bootstrap-italia icon indicating copy to clipboard operation
bootstrap-italia copied to clipboard

Il componente chips non coincide con UI kit Italia

Open zetareticoli opened this issue 1 year ago • 5 comments

Versione di Bootstrap Italia

v2.7.7

Comportamento atteso

L'aspetto visivo (font, spacing, size) e le varianti devono essere le stesse presenti nella scheda del componente presente in UI kit Italia.

In Bootstrap Italia esistono le varianti alte 24px (default) e 32px (lg). In UI Kit Italia esistono le varianti alte 24px (small), 40px (medium) e 48px (large).

Chips in Bootstrap Italia Chips in UI Kit Italia

Comportamento attuale

Le varianti non hanno gli stessi nomi, le scelte di font-size e padding sono differenti. Questo comporta una diversa altezza del componente e dei suoi elementi interni (label, icone).

Possibili soluzioni

Rivedere i valori utilizzati per l'elemento, utilizzando dove possibile variabili locali del componente.

Contesto

zetareticoli avatar Nov 27 '23 18:11 zetareticoli

@zetareticoli concordo con la tua issue. Vi segnalo che era già stata segnalata (scusate il gioco di parole) qui https://github.com/italia/bootstrap-italia/issues/997 la discrepanza, poi chiusa per errore immagino quando è stato fatto un primo fix sulle misure di margine esterno per i gruppi.

Riaprirei la issue originale che conteneva già una parte di discussione @astagi .

Fupete avatar Nov 28 '23 11:11 Fupete

@Fupete riporto la discussione qua dentro, almeno abbiamo un'unica issue. Modificata la precedente per le sole spaziature.

astagi avatar Nov 30 '23 15:11 astagi

Riporto inoltre le considerazioni di @alessandrovecchi

Mia opinione, 40px e 48px mi sembrano molto grandi. Inoltre per una scalatura più corretta sarebbe meglio 24px (small), 36px (medium) e 48px (large).

astagi avatar Nov 30 '23 15:11 astagi

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] avatar Jan 30 '24 02:01 stale[bot]

Stale stai tranquillo. Qui per reminder.

Fupete avatar Jan 30 '24 09:01 Fupete