search

italia / design-comuni-pagine-statiche

Le pagine statiche che compongono il modello di sito web per i comuni
https://italia.github.io/design-comuni-pagine-statiche
BSD 3-Clause "New" or "Revised" License
20 stars 25 forks source link

Template argomento #54

Closed AlessandroVecchi closed 1 year ago

AlessandroVecchi commented 2 years ago

Ciao, La pagina "Argomenti" https://italia.github.io/design-comuni-pagine-statiche/sito/template-argomenti-argomento.html ha un layout diverso dalle altre pagine del modello. Segnali alcuni errori/incongruenze

  1. Il titolo H1 è definito con <h1 class="mb-3 mb-lg-4 title-xxlarge">Sport</h1> a differenze della altre pagine dove è <h1 class="text-black hero-title" data-element="page-name">Amministrazione</h1> Quindi oltre al diverso stile (il font risulta più piccolo, non ho capito se è voluto) manca anche il data-element

  2. Analogamente il testo sotto al titolo è definito con <p class="u-main-black text-paragraph-regular-medium mb-60"> al posto di <div class="hero-text">

  3. è presente un <h2 class="visually-hidden" id="news-details">Dettagli della notizia</h2> che mi sembra fuori contesto e non ho capito a cosa serva.

  4. il primo h3 "Novità" a risoluzione SM (da 576 a 767px) viene nascosto. Questo perché il suo container ha classe <div class="bg-grey-card pt-40 pt-md-100 pb-50"> dovrebbe invece avere una classe "pt-sm-100" ma questa non è stata definita nel framework.

  5. Segnalo infine che gli eventi sono stati messi insieme alle novità sebbene nella coreografia di navigazione la sezione "eventi" sia stata posizionata sotto "vivere il comune" e non sotto "novità". Per logica dovrebbero avere una riga a se stante oppure, volendo tenerli insieme, rinominare la riga in "novità ed eventi" o qualcosa di simile.

Se trovo altro lo aggiungo ;)

zetareticoli commented 2 years ago

Ciao @AlessandroVecchi,

grazie per le tue segnalazioni, sempre molto dettagliate.

Verifichiamo quanto elencato e ti facciamo sapere. Sicuramente i bug saranno presi in carico con urgenza. Quali pagine hai usato per il confronto da te elencato?

AlessandroVecchi commented 2 years ago

Ciao @AlessandroVecchi,

grazie per le tue segnalazioni, sempre molto dettagliate.

Verifichiamo quanto elencato e ti facciamo sapere. Sicuramente i bug saranno presi in carico con urgenza. Quali pagine hai usato per il confronto da te elencato?

Ciao, Le pagine di primo livello (amministrazione, servizi, ecc.)

Guardando adesso velocemente il codice della pagine di dettaglio (notizia, evento) ho visto che il codice dell'h1 è ancora diverso e nella pagina https://italia.github.io/design-comuni-pagine-statiche/sito/template-novita-notizia.html c'è proprio quel h2 che probabilmente è stato riportato per errore nella pagina di argomento.

AlessandroVecchi commented 2 years ago

Sempre sulla pagina argomenti, le sezioni relative ad Amministrazione, Servizi e Documenti sono un po' limitate. Sono presenti solo 3 elementi per riga con un pulsante "vedi tutti" che non si capisce se deve puntare alla pagina generale o all'elenco di tutti gli elementi associati all'argomento.

Sarebbe più funzionale la soluzione già adottata col precedente modello sul comune di Cagliari https://www.comune.cagliari.it/portale/page/it/arte_e_cultura?contentId=ARG10910 Dove è presente una paginazione degli elementi, quindi senza uscire dal contesto, ed un filtro sui risultati (quest'ultimo sicuramente migliorabile).

Una nota sulla sezione "documenti" che era presente come a se stante nell'architettura del modello 1.0 ma che nel 2.0 è stata spostata sotto "amministrazione". In teoria, se la pagina argomento deve elencare le sezioni di primo livello, queste dovrebbero essere: Amministrazione, Novità, Servizi e Vivere il comune. Per ogni sezione di primo livello potrebbe esserci un filtro per quelle di secondo livello. Es. per vivere il comune: "eventi" e "luoghi".

Insomma, direi che un po' tutto questo template andrebbe rivisto....

AlessandroVecchi commented 2 years ago

Altra segnalazione. Le righe delle sezioni amministrazione, servizi e documenti sono costruite con le classi flex invece che sulla griglia di boostrap. Nel caso in cui ci sono 2 elementi anziché 3 essi appaiono disallineati es. immagine

Segnalo inoltre che per card-teaser-wrapper non c'è un componente come per altri elementi del modello ma è statico e scritto per esteso ogni volta che viene usato.

zetareticoli commented 2 years ago

Altra segnalazione. Le righe delle sezioni amministrazione, servizi e documenti sono costruite con le classi flex invece che sulla griglia di boostrap. Nel caso in cui ci sono 2 elementi anziché 3 essi appaiono disallineati es. immagine

Segnalo inoltre che per card-teaser-wrapper non c'è un componente come per altri elementi del modello ma è statico e scritto per esteso ogni volta che viene usato.

In merito a questo ti chiedo la cortesia di aprire una issue dedicata. Grazie 🙏

zetareticoli commented 1 year ago

Chiudo perché risolta in #56