fix issue#85, change @media max-width values for correct render

pdonadeo / e-uscito-joypad

Potente motore di ricerca per sapere se e quando è uscito Joypad, il podcast a tema videoludico di Matteo Bordone (Corri!), Francesco Fossetti (Salta!) e Alessandro Zampini (Spara! per finta). È possibile cercare i titoli recensiti nel podcast e sapere in quale episodio se ne è parlato ed in quale minuto.

https://www.euscitojoypad.it

MIT License

37 stars 5 forks source link

fix issue#85, change @media max-width values for correct render #119

Closed FrancescoPenasa closed 2 days ago

FrancescoPenasa commented 1 month ago

La issue #85 si presentava solo quando la risoluzione orizzontale era di 800px, ho ridotto i valori del @media max-width nel progetto (prima 800px, ora 799px) per essere consistenti con i valori @media min-width (800px)

pdonadeo commented 1 month ago

@FrancescoPenasa ti ringrazio per la patch. Non disponendo di un Lenovo come quello della #85 ho provato la tua patch con Chrome, selezionando un tablet alla risoluzione di 800x600, in questo modo:

È corretto? Vorrei provare in qualche modo la patch prima di fare merge.

Ad ogni modo facendo in questo modo ho ottenuto una versione centrata ma tagliata ai lati del sito, così:

localhost_5000_

Ti risulta questa situazione? Come l'hai testato tu?

agofa commented 1 month ago

Quel tablet ce l'ho io. @pdonadeo collaudo esiste ancora? Se sì potrei testare lì, ma devi ricordarmi tutto che non so più come ci si arriva

FrancescoPenasa commented 1 month ago

@pdonadeo Si è così, il problema è proprio dato dai 800px di larghezza, su qualsiasi device. È vero che questa patch taglia i lati ma la issue era proprio relativa al fatto che a 800px avesse un comportamento particolare, poi se si vuole evitare anche il taglio dei lati va aumentato il valore in px, ma almeno con queste modifiche non c'è più il caso specifico come era descritto nella #85

pdonadeo commented 4 weeks ago

@agofa ok, appena ho tempo faccio merge della patch di @FrancescoPenasa e la metto on line sul sito di staging.

Non so quando riuscirò a farlo perché sono in un periodo complicato e arrivo a sera molto stanco.

Francesco intanto ti chiedo: tu saresti in grado di risolvere anche il problema del "taglio dei bordi", per tutte le risoluzioni?

FrancescoPenasa commented 4 weeks ago

@pdonadeo dovrebbe essere un fix molto simile a quello che ho appena fatto, direi che se non ci sono sorprese posso farlo anche facilmente

pdonadeo commented 4 weeks ago

@FrancescoPenasa taggami qui quando hai fatto: farò merge e metterò on line su staging.

FrancescoPenasa commented 5 days ago

@pdonadeo dovrebbe funzionare ora

pdonadeo commented 5 days ago

@pdonadeo dovrebbe funzionare ora

Nel fine settimana la testo. Intanto ti ringrazio moltissimo e mi spiace non poterla provare subito ma ho poco tempo in questo periodo per i side project.

pdonadeo commented 4 days ago

@FrancescoPenasa @agofa il fix di Francesco è on line qui: https://staging.euscitojoypad.it/

User: joypad Pwd: corrisaltaspara

Fatemi sapere se è tutto OK che lo metto in produzione.

agofa commented 3 days ago

Ho fatto una prova veloce con il tablet. Mi sembra perfetto Domani farò prove più precise

agofa commented 2 days ago

Ho fatto una prova veloce con il tablet. Mi sembra perfetto Domani farò prove più precise

MI rispondo da solo, come un Borghi qualsiasi. A me sembra perfetto ora. Ho provato vari browser su un tablet Lenovo

Unico piccolo appunto ma mi ci mando da solo è che le icone instagram e discord sono un po' troppo attaccate al bordo dell'elenco episodi Ma come detto mi ci sono appena mandato senza passare per il via

immagine

pdonadeo commented 2 days ago

È on line. Grazie a @FrancescoPenasa per il fix e ovviamente ad @agofa per il test.