-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Argomenti in Evidenza - Gestione sfondo, testo e sidebar #590
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mi piace. Aggiungerei una cosa, al momento vedo che rimane lo sfondo colorato quando sono in Modifica e non ci sono argomenti selezionati. Farei sparire lo sfondo anche in Modifica quando non ci sono argomenti selezionati, in modo che si comporti in maniera simile alla View.
@pnicolli @Wagner3UB ho aggiornato la pr come diceva Piero, nascondendo il background se non ci sono card Argomento, inoltre ho aggiunto due prop, una che permette di allineare le chips degli altri argomenti al centro e una seconda che permette di nascondere il pulsante "Vedi tutti" fisso sotto al blocco che punta alla pagina /argomenti, aggiungendoli qui non abbiamo bisogno di customizzare il blocco su Varese, nei nostri attuali io-comune non cambia niente perchè di default sono false. |
@pnicolli Vedi se ti convince fare la modifica per centrare gli argomenti su io-Comune. @SaraBianchi non sapeva che era una custom solo per Varese. |
Mi sta bene metterlo se è un parametro e se il default è false, cioè se il default è che rimane com'è prima. |
@pnicolli Infatti è così, ho sistemato i conflitti, se è tutto ok si può mergiare |
*Vedere sulla US il video-test