feat! :update bootstrap-italia to 2.8.4 #647
Conversation
deodorhunter
requested review from
pnicolli,
SaraBianchi,
giuliaghisini and
Wagner3UB
probabilmente alcune cose fatte appositamente da noi per l'accessibilità andranno tolte se di base i componente di design-react-kit lo fanno.. c'è da rifare un giro generale |
Alcune cose tipo i focus--mouse le hanno messe, ma rotte, gia' sistemate. Ho fatto un test al volo con l'estensione di axe e non sono inciampata in cose strane, ma tocca rigirarselo per davvero e io qui tecnicamente non avrei piu' tempo sul task. Di sicuro macera qui per la prossima minor pps: ocio che design-react-kit === bootstrap-italia e' una misconception: la versione che abbiamo noi e' tecnicamente ferma alla 2.6.x di bootstrap italia |
| @@ -82,7 +82,7 @@ const ListingBody = React.memo( | |||
| isSearchBlockResults ? '' : 'full-width' | |||
| }`; | |||
| } else { | |||
| return `py-5 ${bg_color} ${ | |||
There was a problem hiding this comment.
perchè hai tolto il colore di sfondo?
There was a problem hiding this comment.
perche' era applicato quando non era spuntato "mostra lo sfondo", che e' sbagliato. E' il padre che gestisce lo sfondo, non il template listing mostrato
There was a problem hiding this comment.
Facendo qualche prova però senza questa classe non viene mai mostrato, e se la ripristino, viene mostrato solo quando è spuntato "mostra lo sfondo"
| line-height: 1.555; | ||
| // line-height: 1.555; ? Already set by bootstrap-italia | ||
| } | ||
| blockquote:before { |
There was a problem hiding this comment.
questa regola sui blockquote siamo sicuri di volerla mettere in questo file? non c'è un file dedicato?
Lascio un paio di screen di una pagina campione per report automatizzato wcag AA con axe. Preso la pagina piu' grossa che abbiamo su io-comune Sono uguali, nulla di nuovo e' stato introdotto ne in bene ne in male a quanto sembra |
| @@ -3,7 +3,7 @@ | |||
| * @module components/theme/View/ServizioView | |||
| */ | |||
|
|
|||
| import React, { createRef } from 'react'; | |||
| import React, { createRef, useRef } from 'react'; | |||
There was a problem hiding this comment.
Rimuovi dall'import createRef
| @@ -82,7 +82,7 @@ const ListingBody = React.memo( | |||
| isSearchBlockResults ? '' : 'full-width' | |||
| }`; | |||
| } else { | |||
| return `py-5 ${bg_color} ${ | |||
| return `py-5 ${ | |||
|
Update: in stand-by senza data di fine in vista, dipendente dagli aggiornamenti a volto18 e a design-react-kit |
Hanno solo fatto piccoli breaking changes all'intera tipografia, spacing e altre amenita'. Controllare i prodotti io-cittadino e io-prenoto, lascio in draft.
Visto che hanno introdotto "changes per accessibilita'", bisogna rifare un test automatizzato anche