italia / design-comuni-wordpress-theme

Tema Wordpress per i siti dei comuni italiani
GNU Affero General Public License v3.0
33 stars 33 forks source link

Notizie nella home sovrapposte #376

Closed Cam-al closed 1 year ago

Cam-al commented 1 year ago

Prerequisiti

Descrivi il problema

Ho notato solo oggi che la notizia in evidenza nella home tende ad andare sopra le altre notizie coprendo la categoria, sicuramente questo prima non accadeva ma non so se questo errore possa essere collegato all'ultimo aggiornamento. Pensavo di avere modificato qualcosa io nel CSS ma non ho trovato nulla, per curiosità sono andato a vedere il comune di Sedriano e ho notato lo stesso errore di visualizzazione.

Includi link o screenshot

No response

Su quale sisitema operativo stai riscontrando il problema?

Windows

Su quale browser stai riscontrando il problema?

Qualsiasi

Che versione del tema stai utilizzando?

1.7.3

enrimk commented 1 year ago

L'html della home di WP è radicalmente diverso da quello nelle statiche. E tra parentesi, la disposizione nelle statiche non mi pare meglio, anzi (le 3 card in evidenza dentro section#calendario? Perché?). Quindi un eventuale difetto nel css non viene testato ed evidenziato.

Per ora, io ho semplicemente "risolto" così:

/* Fix comuni.css */
.card-wrapper.card-overlapping {
    z-index: 1;
}
Cam-al commented 1 year ago

si pure io ho risolto così, ma prima si vedeva bene e non ho capito da quando è diventato cosi.

enrimk commented 1 year ago

Non credo che sia stata una modifica nel codice, non mi pare che recentemente ci siano stati aggiornamenti delle parti che hanno influenza su quel punto. È più facile che sia cambiato qualcosa nel browser. Infatti credo che solo Chrome (ed Edge) manifestino il problema, con Firefox mi sembra che non ci sia.

Dev'essere qualcosa di un po'... arcano, legato al rendering dei vari "layer" nel browser. Non sono sicuro, ma mi sa che ci sia qualche connessione con il fatto che il flexbox nella novità in evidenza usa "order" per riposizionare i suoi elementi nelle viste mobile. Magari questo rimanda o anticipa il painting dei vari blocchi e ne altera l'"impilamento" e lo stacking context, chissà.

fabricius4 commented 1 year ago

Non credo che sia stata una modifica nel codice, non mi pare che recentemente ci siano stati aggiornamenti delle parti che hanno influenza su quel punto. È più facile che sia cambiato qualcosa nel browser. Infatti credo che solo Chrome (ed Edge) manifestino il problema, con Firefox mi sembra che non ci sia.

Dev'essere qualcosa di un po'... arcano, legato al rendering dei vari "layer" nel browser. Non sono sicuro, ma mi sa che ci sia qualche connessione con il fatto che il flexbox nella novità in evidenza usa "order" per riposizionare i suoi elementi nelle viste mobile. Magari questo rimanda o anticipa il painting dei vari blocchi e ne altera l'"impilamento" e lo stacking context, chissà.

anche su browser Opera da lo stesso problema ma con il fix di @enrimk si risolve

enrimk commented 1 year ago

flexbox nella novità altera l'"impilamento" e lo stacking context

Infatti una "correzione" alternativa potrebbe essere così:

#novita-in-evidenza + .row {
    isolation: isolate;
}
zetareticoli commented 1 year ago

L'html della home di WP è radicalmente diverso da quello nelle statiche

Questo accade perché l'implementazione del template su WP ha richiesto dei cambiamenti rispetto alla versione statica.

zetareticoli commented 1 year ago

@Cam-al potresti allegare screeshots del bug?

enrimk commented 1 year ago

L'html della home di WP è radicalmente diverso da quello nelle statiche

Questo accade perché l'implementazione del template su WP ha richiesto dei cambiamenti rispetto alla versione statica.

Si può prendere in considerazione di "sincronizzare" le home aggiornando le statiche e riportando là alcune delle modifiche di WP? Per certi versi la struttura di quella zona della home WP sembra anche più "logica".

enrimk commented 1 year ago

potresti allegare screeshots del bug?

È possibile che si sia risolto da sé - o meglio, con gli ultimi aggiornamenti dei browser, da qualche giorno. Infatti io non lo vedo più in nessun sito, compreso Sedriano. A questo punto, sarà stato davvero solo un bug nel rendering engine.

Cam-al commented 1 year ago

@zetareticoli ho provato a commentare

/* Fix comuni.css */
.card-wrapper.card-overlapping {
    z-index: 1;}

e adesso non si verifica più l'errore, come diceva @enrimk potrebbe essere qualcosa nel browser