Skip to content
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

Liste immagini: codice di esempio non aggiornato #1279

Open
1 task done
arturu opened this issue Nov 19, 2024 · 5 comments
Open
1 task done

Liste immagini: codice di esempio non aggiornato #1279

arturu opened this issue Nov 19, 2024 · 5 comments
Assignees
Labels
docs Issue e PR che riguardano la documentazione

Comments

@arturu
Copy link
Contributor

arturu commented Nov 19, 2024

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

  • Ho verificato e non esiste

Versione della libreria

2.11.*

Cosa

Questo esempio https://italia.github.io/bootstrap-italia/docs/organizzare-i-contenuti/liste-di-immagini/#standard non è aggiornato con il codice della v >= 2.11.*

Perché

Perché bisogna aggiornare il codice di questo esempio

Contesto

No response

Altro

No response

@astagi
Copy link
Member

astagi commented Nov 20, 2024

Ciao @arturu che intendi per non aggiornato? nella documentazione gli esempi utilizzano l'ultima versione buildata

@astagi astagi added the need info Servono maggiori informazioni label Nov 20, 2024
@arturu
Copy link
Contributor Author

arturu commented Nov 20, 2024

Nell'esempio https://italia.github.io/bootstrap-italia/docs/organizzare-i-contenuti/liste-di-immagini/#tipi-di-griglia l'elemento ha questo codice (A):

<div class="it-grid-item-wrapper">
  <a href="#">
    <div class="img-responsive-wrapper">
      <div class="img-responsive">
        <div class="img-wrapper"><img src="https://placehold.co/660x300/ebebeb/808080/?text=Immagine" alt="descrizione immagine"></div>
      </div>
    </div>
  </a>
</div>

invece in tutti gli altri esempi l'elemento ha questo codice (B)

<div class="it-grid-item-wrapper">
  <a href="#">
    <figure class="figure img-full w-100">
      <img src="https://placehold.co/660x300/ebebeb/808080/?text=Immagine" class="figure-img img-fluid rounded" alt="Un'immagine generica segnaposto con angoli arrotondati in una figura.">
    </figure>
  </a>
</div>

quindi è possibile che codice (A) sia obsoleto e vada aggiornato usando codice (B).

Provo a fare una pull, forse si capisce meglio; se non va bene rifiutala tranquillamente :)

arturu added a commit to arturu/bootstrap-italia that referenced this issue Nov 20, 2024
@astagi
Copy link
Member

astagi commented Nov 20, 2024

Ah perdonami, ho capito @arturu ... se non ricordo male in realtà è stato scelto di mantenere entrambe le possibilità di definire l'immagine, non è un errore. Possiamo magari specificare meglio questa cosa invece di cambiare il codice.. @Fupete che ne dici?

@arturu
Copy link
Contributor Author

arturu commented Nov 20, 2024

Ah perdonami, ho capito @arturu ... se non ricordo male in realtà è stato scelto di mantenere entrambe le possibilità di definire l'immagine, non è un errore. Possiamo magari specificare meglio questa cosa invece di cambiare il codice.. @Fupete che ne dici?

Ah ok, non l'avevo capito allora, scusa :)

@astagi
Copy link
Member

astagi commented Nov 20, 2024

Figurati @arturu nessun problema, lascio aperta in attesa del da farsi.

@Fupete Fupete added docs Issue e PR che riguardano la documentazione and removed need info Servono maggiori informazioni labels Dec 2, 2024
@Fupete Fupete moved this from 🆕 New to 📋 Backlog in Design system .italia 🇮‍🇹 Dec 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Issue e PR che riguardano la documentazione
Projects
Status: 📋 Backlog
Development

Successfully merging a pull request may close this issue.

3 participants