Skip to content
This repository was archived by the owner on Oct 10, 2023. It is now read-only.

Conversation

@giogardin
Copy link
Owner

@giogardin giogardin commented Feb 21, 2023

Ho fatto qualcosina riguardo la fantomatica pubblicità. Ho deciso di stare KISS: la "pubblicità", per Soundstage, sarà la possibilità di impostare i film in evidenza.

  • aggiornato lo schema DB per memorizzare lo stato "in evidenza" (boolean)
  • aggiornato PHP per gestire il nuovo campo
  • aggiornati gli scheletri HTML di card (cerca.html) e pagina del film (film.html) per accogliere un badge di stato
  • aggiornato il form di aggiunta/modifica dei film (gest_film.*)

Per testare occorre aggiornare lo schema del DB. È una modifica talmente banale che secondo me la potete fare direttamente da PMA (senza reimportare tutto il dump, che tra l'altro non ho nemmeno aggiornato).

Vi chiedo di controllare:

  • aggiunta di film
  • modifica di film
  • colori (funzionano, ma se volete cambiarli siete liberi di proporne di nuovi) @giovannifil-64
  • accessibilità (ho fatto le cose con criterio, ma è sempre bene controllare di nuovo)

Bonus: aggiungere un paragrafo in About per spiegare la cosa agli sponsor pieni di cash.

Una volta mergiato, close #421.

@giogardin giogardin added the enhancement New feature or request label Feb 21, 2023
@giogardin giogardin self-assigned this Feb 21, 2023
@giovannifil-64
Copy link
Collaborator

giovannifil-64 commented Feb 21, 2023

L'aggiunta/modifica dello stato funziona senza problemi. Unica cosa lo stile dell'icona è sfasato (non ho fatto modifiche al codice)
Schermata 2023-02-21 alle 15 22 39

Inoltre, collezioni e persone risultano tutti come in evidenza, immagino si tratti di un bug
Schermata 2023-02-21 alle 15 22 53

Inoltre, non mi è chiaro il comportamento dell'evidenza, non dovrebbe comparire (praticamente) in tutte le ricerche?

@giogardin
Copy link
Owner Author

L'aggiunta/modifica dello stato funziona senza problemi. Unica cosa lo stile dell'icona è sfasato

Sounds like un problema di caching

Inoltre, collezioni e persone risultano tutti come in evidenza, immagino si tratti di un bug

Sì, fixo

Inoltre, non mi è chiaro il comportamento dell'evidenza, non dovrebbe comparire (praticamente) in tutte le ricerche?

L'ho implementata come un semplice badge. Metterla in tutte le ricerche non sarebbe molto user friendly, però magari posso modificare la query per mostrare i film in evidenza in cima ai risultati di ricerca

@giovannifil-64
Copy link
Collaborator

giovannifil-64 commented Feb 21, 2023

L'aggiunta/modifica dello stato funziona senza problemi. Unica cosa lo stile dell'icona è sfasato

Sounds like un problema di caching

Infatti, risolto

Inoltre, non mi è chiaro il comportamento dell'evidenza, non dovrebbe comparire (praticamente) in tutte le ricerche?

L'ho implementata come un semplice badge. Metterla in tutte le ricerche non sarebbe molto user friendly, però magari posso modificare la query per mostrare i film in evidenza in cima ai risultati di ricerca

Eventualmente la si potrebbe pensare in questo modo, se poi si volesse andare su qualcosa di un pelino più avanzato e complicato, fare mi modo compaia un film a caso tra quelli in evidenza come primo elemento della ricerca

@giovannifil-64
Copy link
Collaborator

Altro piccolo glitch, la dimensione della badge in base alla lunghezza del film

Schermata 2023-02-21 alle 15 44 51

Schermata 2023-02-21 alle 15 44 59

@giogardin
Copy link
Owner Author

Ok, sistemata la larghezza del badge e impostata la query in modo da avere i risultati in evidenza all'inizio.

@giovannifil-64
Copy link
Collaborator

Ho visto, perfetto

@giovannifil-64
Copy link
Collaborator

Per ora non vedo alti grossi problemi, segnalo le ultime due cose poi penso vada tutto bene, aspettando il commento degli altri.
Un film, se è in evidenza, non compare come tale nella collezione
Schermata 2023-02-21 alle 15 56 23
Schermata 2023-02-21 alle 15 56 19

I film in evidenza li mostriamo anche quando un utente esegue la query come primi risultati? O rischiamo di incasinare troppo?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Implementare pubblicità

3 participants