-
Notifications
You must be signed in to change notification settings - Fork 161
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
feat: aggiunto componente point list #1334
base: main
Are you sure you want to change the base?
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ciao @arturu , grazie innanzitutto per questo contributo.
Modificherei lo scope del componente, eliminando il contesto specifico di "Lista di eventi".
Se, infatti, è disponibile la variante a step, in tal caso questo componente potrebbe essere utile per descrivere procedure senza vincoli temporali.
Forse la parola "eventi" richiama alla mente un qualcosa di organizzato.
Potremmo provare ad astrarlo maggiormente.
@zetareticoli grazie per il feedback, ci penso un po' e cerco di trovare un nome più generico. Nel frattempo a te è venuta in mente qualche idea? |
Più ci penso e più è oggettivamente anche questo un componente timeline a livello semantico (seppur diverso da quello di presentazione che già abbiamo). "Lista timeline" o "Lista cronologica"? Oppure, e forse qui potrebbe essere interessante per semplificare invece di aggiungere, si potrebbe pensare di unire vecchio e nuovo? Mi spiego, questo diventa il nuovo componente timeline che sostituisce il precedente in toto. Ma lo fa in modo da poter essere usato anche per realizzare il precedente layout (che ne diventa solo una variante)... partendo cioè dall'esempio complesso del nuovo componente https://bootstrap-italia-fld2lxrhn-dip-trasformazione-digitale.vercel.app/docs/organizzare-i-contenuti/liste-di-eventi/#esempio-complesso si aggiunge un esempio ALT per avere la linea centrale e se vogliamo anche i PIN vecchio stile con i mesi come la timeline attuale https://bootstrap-italia-fld2lxrhn-dip-trasformazione-digitale.vercel.app/docs/componenti/timeline/#esempio. Secondo me potrebbe semplificare la cosa. In effetti entrambi i componenti sennò farebbero la stessa cosa non vi pare? Sarebbe forse una direzione possibile? |
Se vogliamo eliminare il riferimento temporale (cronologia/timeline) concordo che è meglio rimanga a sé. E in quel caso vi proporrei di ragionare su un nome come "Lista di processo". O qualcosa del genere che questo suona male, ma rende l'idea. Focalizzato sul presentare azioni sequenziali da fare, con o senza data. |
Obiettivamente il componente presenta una "lista di contenuti ordinati cronologicamente" a prescindere se la data è esplicitata o meno. Quando la data è esplicitata è più simile al componente Timeline, invece quando la data non è esplicita, è simile alle liste tipo step-by-step.
Quindi potrebbe essere: Andare a modificare la timeline è abbastanza impegnativo a livello di tempo... :( |
Descrizione
Fixes #1324
Ho aggiunto il componente, penso che sia necessaria una review, specialmente dal punto di vista dell'accessibilità.
Nello specifico ho aggiunto il componente point-list
come variante di timelinenella pagina "Organizzare i contenuti" -> "Liste di eventi", logicamente se si pensa che non è nel posto opportuno si modifica.Ho fatto delle modifiche al css in quanto c'erano alcuni problemini con il bullet, in base alle dimensioni del contenuto ed agli elementi del componente il bullet era fuori posto. Penso che ho raggiunto un buon compromesso, come sempre si accettano consigli per migliorare :)
Checklist