-
Notifications
You must be signed in to change notification settings - Fork 215
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
Layout orizzontale e miglioramento verticale home #596
Conversation
- Possibilità di attivare il layout orizzontale dei contenuti relativi alle novità in pagina iniziale (un tipo di contenuto per riga) - Possibilità di configurare un numero personalizzato di circolari da visualizzare in pagina iniziale - Possibilità di configurare un numero personalizzato di eventi da visualizzare in pagina iniziale
Chiediamo a tutti coloro che avevano aperto una issue in merito di testare il miglioramento e poter aggiungere un riscontro |
- I contenuti in evidenza in pagina iniziale vengono aggiunti e non sostituiti ai contenuti caricati automaticamente - Gli articoli visualizzati in evidenza vengono esclusi dal caricamento automatico
In primis complimenti per il lavoro ;) |
Grazie @sviluppoRobyone per il prezioso contributo! Abbiamo notato che cambiando la visualizzazione (da verticale a orizzontale e viceversa) cambia l'ordine degli eventi che vengono visualizzati e non è sempre rispettato il numero di eventi indicati da visualizzare in home. Ho fatto la prova inserendo "5" in backend alla voce "Eventi da mostrare in home". Ti volevo poi chiedere una cosa: ritieni sia opportuno per le scuole vedere al primo posto l'evento che si verifica oggi? |
Eccomi, in effetti la visualizzazione orizzontale ha già applicato la PR #519 , per quello si vedono due cose diverse dove non è applicata. Ho pensato alla ormai problematica questione degli eventi in pagina iniziale, una soluzione definitiva e migliorativa di quell'area potrebbe essere questa che riporto in immagine (mostra il layout orizzontale ma se è questa la strada lo applichiamo facilmente anche all'altro) A questo punto ci sarebbe coerenza anche con la pagina interna del calendario. Poi se vogliamo, possiamo aggiungere ulteriore opzione per decidere se mostrare o non mostrare gli eventi in corso in pagina iniziale |
Dimenticavo di rispondere alla tua domanda, a livello logico, se vedo gli eventi in corso e gli eventi prossimi assieme, direi di sì, è opportuno rispettare l'ordine cronologico, per non creare confusione. |
@sviluppoRobyone mi sembra un ottimo lavoro. State aspettando ulteriori feedback o posso concludere la review? |
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.
Ci sono modifiche da riportare eventualmente sul repo dei templates statici?
In caso, sarebbe opportuno aprire una PR anche lì riportando il template alternativo.
Se si può riportare un template alternativo, vediamo di aggiungere una PR anche lì. Più che altro procediamo se c'è accordo anche con la modifica degli eventi come da proposta (attualmente manca il codice per produrre il risultato dell'immagine). |
@sviluppoRobyone ti rispondo per punti così non tralascio nulla:
|
1- Confermiamo che l'intenzione è quella di avere stesso comportamento orizzontale e verticale (in questo caso per averlo, è necessario applicare entrambe le PR (penso che a questo punto con la prossima modifica degli eventi, mettiamo tutto qui così diventa più chiaro) 2- Rimane da capire se vogliamo inserire un'opzione oppure indicare un'unica strada, quella di includere gli eventi correnti senza possibilità di escluderli (penso però che a questo punto dato che ci siamo mettere un'opzione non sia un problema) 3- Esatto, il link passa alla pagina degli eventi in corso / futuri (qualora invece non ci siano eventi, si potrebbe lasciare il link all'archivio) |
Per me, possiamo procedere. |
Perfetto! |
…e-archivio' into layout-orizzontale-home
Modifiche su eventi home e riordino
Ora ci dovremmo essere, a questo punto attenderei vostro riscontro anche su un altra proposta (#597 ) così da creare un'unica PR se accettata |
Esigenza legata anche a contenuti di albo online e amministrazione trasparente che ad oggi non potevano essere messi in evidenza in pagina iniziale
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.
@sviluppoRobyone sono state riportate tutte le modifiche previste anche in #597 (ormai chiusa) ?
Confermo |
Aggiornamento istruzioni Configurazione
Descrizione
Dopo l'aggiornamento è necessario entrare nella scheda Configurazione > Home ed effettuare nuovamente selezione e salvataggio del valore dell'opzione della visualizzazione automatizzata. Abbiamo comunque fatto in modo che nella parte pubblica ci sia retrocompatibilità.
Fixes #570 #536 #466 #294 #599 #598 #610
Checklist