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

Modifica comportamento contenuti in evidenza (home) #597

Conversation

sviluppoRobyone
Copy link
Contributor

@sviluppoRobyone sviluppoRobyone commented Jan 17, 2024

Descrizione

  • 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

Checklist

- 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
@sviluppoRobyone sviluppoRobyone changed the title Modifica comportamento contenuti in home-evidenza Modifica comportamento contenuti in evidenza (home) Jan 17, 2024
@sviluppoRobyone
Copy link
Contributor Author

Abbiamo chiesto questa modifica perché parte degli istituti che collaborano con noi chiedono di poter fissare dei contenuti senza tuttavia rinunciare al caricamento automatico di quelli non in evidenza.

@zetareticoli zetareticoli self-requested a review January 22, 2024 17:48
@zetareticoli zetareticoli added the enhancement New feature or request label Jan 22, 2024
Copy link
Member

@zetareticoli zetareticoli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Con questa modifica, la sezione in evidenza quindi può ospitare contenuti fissi e automatici insieme?

@sviluppoRobyone
Copy link
Contributor Author

sviluppoRobyone commented Jan 23, 2024

Esatto, vi riporto i due screen di esempio.
Sarebbe bello presentare meglio la pagina in evidenza, al momento abbiamo tenuto lo stile com'era prima.

Aggiungo altresì che sarebbe bello scegliere una fascia di colore leggermente diverso per sottolineare, ma al momento non abbiamo avuto idee in merito percorribili, rimaniamo a disposizione per suggerimenti

PARTE ADMIN
image

RISULTATO
image

Copy link
Member

@zetareticoli zetareticoli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nel caso ci siano card con più o meno contenuti, le card dovrebbero comunque avere tutte la stessa altezza. Riuscite a integrare questo fix?

@sviluppoRobyone
Copy link
Contributor Author

Già risolto nella PR #596 stamattina, se questa modifica è fattibile, per evitare confusione, porterei queste modifiche direttamente nell'altra

@zetareticoli
Copy link
Member

Già risolto nella PR #596 stamattina, se questa modifica è fattibile, per evitare confusione, porterei queste modifiche direttamente nell'altra

Perdonami, non ho capito. Vuoi riportare tutte le modifiche di questa PR in #596?

@sviluppoRobyone
Copy link
Contributor Author

sviluppoRobyone commented Jan 23, 2024

Esatto, dato che riguarda sempre la pagina iniziale (non prevedendo per l'altra PR tutti i cambiamenti che poi sono stati fatti, le avevamo divise, ma per chiarezza e per evitare che qualche modifica non venga fatta non prevedendo entrambe le funzionalità accettate, forse sarebbe meglio unirle)

@zetareticoli
Copy link
Member

@sviluppoRobyone ok, vedi pure tu di unificarle in un'unica PR e chiudi quella che non serve quando concluso, così sappiamo cosa portare in merge

@sviluppoRobyone
Copy link
Contributor Author

Modifiche inserite nella PR #596

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants