Skip to content

Componente Point list #1324

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

Open
1 task done
arturu opened this issue Jan 17, 2025 · 10 comments · May be fixed by #1334
Open
1 task done

Componente Point list #1324

arturu opened this issue Jan 17, 2025 · 10 comments · May be fixed by #1334
Assignees
Labels
feature Issue e PR che riguardano nuove funzionalità

Comments

@arturu
Copy link
Contributor

arturu commented Jan 17, 2025

Esiste già una discussione sul tema che ti interessa, o su un tema simile?

  • Ho verificato e non esiste

Versione della libreria

2.*

Cosa

Il componente non viene citato nelle pagine di documentazione.

Si capisce che è un componente non documentato in quanto esiste il file point-list.scss e viene usato nei vari modelli come quello delle scuole e dei comuni.

Perché

Penso che se esiste un componente debba essere documentato.

Contesto

No response

Altro

@Fupete
Copy link
Contributor

Fupete commented Jan 17, 2025

Ottimo catch @arturu.
È collegato anche a questa (seppur tangente) italia/design-ui-kit#132

cc @astagi @zetareticoli

@Fupete Fupete added the question Domande e curiosità label Jan 17, 2025
@astagi
Copy link
Member

astagi commented Jan 17, 2025

@Fupete @zetareticoli @arturu direi che possiamo inserirlo nella documentazione delle liste.

@arturu ti senti di provare a fare una PR per portare alla luce questo componente?

@arturu
Copy link
Contributor Author

arturu commented Jan 18, 2025

@astagi yeah sì mi piacerebbe, grazieee :) :) nel tempo libero mi ci dedico volentieri

Intanto ho dato un'occhiata a italia/design-ui-kit#132 e direi che:

  1. il componente per le ASL che cita @zetareticoli in #132 è già pronto — è un accordion con una classe nel titolo per fare il bordo bianco e una classe nel pannello collassabile per fare il bordo grigio a sinistra — quindi "lo toglierei dal calderone" così da ridurre la complessita; quello dei comuni penso che rientra in questa casistica.
  2. il componente step-by-step che cita @ilbassa sembra simile a point list, con la differenza che non viene dato un ordine temporale con date ma con passi.

Quindi che ne dite di trattare "point-list" come una variante di timeline? Timeline in effetti è una lista di contenuti ordinati temporalmente.

Queste sono delle prime considerazioni fatte di getto, come mi ritaglio un po' di tempo ci ragiono meglio.

@Fupete Fupete added feature Issue e PR che riguardano nuove funzionalità and removed question Domande e curiosità labels Jan 20, 2025
@Fupete Fupete moved this from 🆕 New to 🏗 In progress in Design system .italia 🇮‍🇹 Jan 20, 2025
arturu added a commit to arturu/bootstrap-italia that referenced this issue Jan 23, 2025
@zetareticoli
Copy link
Member

Penso che sia meglio avere due componenti diversi.

Le varianti vanno intese come piccole variazioni pur rimanendo nello stesso contesto.

In questo caso, timeline e l'eventuale nuovo cmp sarebbero molto diversi tra loro.

arturu added a commit to arturu/bootstrap-italia that referenced this issue Jan 23, 2025
arturu added a commit to arturu/bootstrap-italia that referenced this issue Jan 23, 2025
arturu added a commit to arturu/bootstrap-italia that referenced this issue Jan 23, 2025
arturu added a commit to arturu/bootstrap-italia that referenced this issue Jan 23, 2025
@arturu arturu linked a pull request Jan 23, 2025 that will close this issue
5 tasks
@arturu
Copy link
Contributor Author

arturu commented Jan 23, 2025

arturu added a commit to arturu/bootstrap-italia that referenced this issue Jan 23, 2025
arturu added a commit to arturu/bootstrap-italia that referenced this issue Jan 23, 2025
@arturu
Copy link
Contributor Author

arturu commented Jan 23, 2025

Ho fatto la modifica, ho aggiunto una nuova pagina Organizzare i contenuti / Liste di eventi

@astagi
Copy link
Member

astagi commented Jan 23, 2025

Grazie @arturu 🙌

arturu added a commit to arturu/bootstrap-italia that referenced this issue Jan 23, 2025
arturu added a commit to arturu/bootstrap-italia that referenced this issue Jan 23, 2025
@Fupete Fupete moved this from 🏗 In progress to 👀 In review in Design system .italia 🇮‍🇹 Feb 14, 2025
@Fupete
Copy link
Contributor

Fupete commented Feb 24, 2025

Ciao @arturu,

abbiamo fatto review della tua ottima PR. Allora, proseguiremmo così:

  1. consideriamo questo "point-list" parte del più ampio componente Timeline. Magari, in prospettiva, definendo questa come una ulteriore variante complessa di una point-list con card alternate tra destra e sinistra e possibilità di avere titoli/mesi sulla riga. Da valutare lo spessore riga e le varianti colore.
  2. Se riesci a riportare tu tutto sotto la documentazione di Timeline (come nella prima ipotesi che avevi fatto) poi magari facciamo un giro noi a sistemare i testi della pagina di documentazione. Unico appunto: per ora eviteremmo di usare nomenclatura tipo "step-by-step" per evitare di fare confusione con il componente Stepper.
  3. Prima di mergiare in produzione ci sarà da fare l'equivalente componente in UI Kit Italia, magari recuperando e sistemando il lavoro presente nei Figma dei Comuni. Su questo ci penseremo noi ma se vuoi procedere benvenga.

Facci sapere cosa ne pensi.
Daniele T.

cc @astagi

@arturu
Copy link
Contributor Author

arturu commented Feb 28, 2025

Ciao @Fupete

wow grazie :) :)

Ok, quindi sposto tutto nella pagina della Timeline (come da prima ipotesi), ed evito di usare il termine step-by-step. Certo, appena mi ritaglio il tempo il questi giorni lo faccio con molto piacere.

Per quanto riguarda il punto 3. non è nelle mie "corde" usare figma, non saprei dove mettere le mani :(

Ciao :)

arturu added a commit to arturu/bootstrap-italia that referenced this issue Mar 16, 2025
arturu added a commit to arturu/bootstrap-italia that referenced this issue Mar 16, 2025
@arturu
Copy link
Contributor Author

arturu commented Mar 16, 2025

@Fupete Penso che ci sono

1. consideriamo questo "point-list" parte del più ampio componente Timeline. Magari, in prospettiva, definendo questa come una ulteriore variante complessa di una point-list con card alternate tra destra e sinistra e possibilità di avere titoli/mesi sulla riga. Da valutare lo spessore riga e le varianti colore.

Ho modificato i testi in questa direzione, spero non mi sia sfuggito nulla

2. Se riesci a riportare tu tutto sotto la documentazione di Timeline (come nella prima ipotesi che avevi fatto) poi magari facciamo un giro noi a sistemare i testi della pagina di documentazione. Unico appunto: per ora eviteremmo di usare nomenclatura tipo "step-by-step" per evitare di fare confusione con il componente Stepper.

Fatto. Per quanto riguarda "step-by-step" ho usato una terminologia diversa per in modo che non ci sia confusione

3. Prima di mergiare in produzione ci sarà da fare l'equivalente componente in UI Kit Italia, magari recuperando e sistemando il lavoro presente nei Figma dei Comuni. Su questo ci penseremo noi ma se vuoi procedere benvenga.

Figma non lo so usare, mi trovo più a mio agio con un editor e un terminale.

cc @astagi

Ciao e grazie :)

arturu added a commit to arturu/bootstrap-italia that referenced this issue Mar 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Issue e PR che riguardano nuove funzionalità
Projects
Status: 👀 In review
Development

Successfully merging a pull request may close this issue.

4 participants