Skip to content
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

Merged
merged 10 commits into from
Apr 8, 2024

Conversation

Wagner3UB
Copy link

*Vedere sulla US il video-test

  • rimosso sfondo de non ci sono card
  • sidebar sistemata per non avere l'obbligatorietà di avere un card
  • Descrizione per info in più sulla sidebar
  • Titolo nero se non c'è lo sfondo
  • gestione spazi se non ci sono card
  • Aggiunte classi per identificazione

@Wagner3UB Wagner3UB requested a review from pnicolli March 15, 2024 14:53
@Wagner3UB Wagner3UB self-assigned this Mar 15, 2024
Copy link
Member

@pnicolli pnicolli left a 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.

@SaraBianchi
Copy link

@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.

Screenshot 2024-03-22 alle 10 46 30 Screenshot 2024-03-22 alle 10 45 38

@SaraBianchi SaraBianchi requested a review from pnicolli March 22, 2024 10:11
@SaraBianchi SaraBianchi self-assigned this Mar 22, 2024
@Wagner3UB
Copy link
Author

@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.

@pnicolli
Copy link
Member

@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.

@SaraBianchi
Copy link

@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

@pnicolli pnicolli merged commit 0215448 into main Apr 8, 2024
3 checks passed
@pnicolli pnicolli deleted the us50242-argomenti-in-evidenza branch April 8, 2024 10:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants