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

Didascalie nelle liste di immagini #1078

Open
cfabry opened this issue Feb 28, 2024 · 4 comments · May be fixed by #1150
Open

Didascalie nelle liste di immagini #1078

cfabry opened this issue Feb 28, 2024 · 4 comments · May be fixed by #1150
Assignees
Labels
a11y Issue e PR riguardanti accessibilità e dintorni

Comments

@cfabry
Copy link
Collaborator

cfabry commented Feb 28, 2024

Versione di Bootstrap Italia

2.8.2

Comportamento atteso

Le didascalie nelle liste di immagini devono essere inserite con semantica <figure> <figcaption> per creare correlazione tra i due contenuti.

Comportamento attuale

Ora sono gestite semplicemente con <span> ossia senza correlazione.

Possibili soluzioni

Bisogna riportare la semantica corretta come giustamente abbiamo fatto in Immagini.

@cfabry cfabry added the a11y Issue e PR riguardanti accessibilità e dintorni label Feb 28, 2024
@zetareticoli zetareticoli self-assigned this Jul 15, 2024
@zetareticoli zetareticoli added this to the M1-24 milestone Jul 15, 2024
@zetareticoli zetareticoli linked a pull request Jul 23, 2024 that will close this issue
5 tasks
@zetareticoli
Copy link
Member

zetareticoli commented Jul 23, 2024

Ciao @cfabry , dopo una prima analisi, ho riscontrato un problema con la struttura HTML utilizzata per l'immagine con didascalia in overlay:

<div class="it-grid-item-wrapper it-grid-item-overlay">
      <a href="#">
        <div class="img-responsive-wrapper">
          <div class="img-responsive">
            <div class="img-wrapper"><img src="https://via.placeholder.com/660x300/ebebeb/808080/?text=Immagine" alt="descrizione immagine"></div>
          </div>
        </div>
        <span class="it-griditem-text-wrapper">
          <span class="it-griditem-text">Didascalia</span>
          <svg class="icon"><use href="/bootstrap-italia/dist/svg/sprites.svg#it-code-circle"></use></svg>
        </span>
      </a>
    </div>

Questo codice (e anche il relativo CSS) presuppone che la didascalia sia allo stesso livello del contenitore dell'immagine, al quale viene applicato poi l'overlay tramite lo pseudo-element ::after.

Dovendo invece utilizzare <figcaption>, questo deve essere dentro al tag <figure>, al livello di <img> che non accetta pseudo-elements.

Una soluzione si può trovare riscrivendo da zero il codice CSS, oltre alla struttura HTML, solo per il caso con overlay.

Negli altri casi, invece, funziona bene con le classi già esistenti in BS Italia.

/cc @Fupete

@zetareticoli
Copy link
Member

@Fupete per risolvere questa issue va riscritto il codice HTML della versione con overlay. Da capire se procedere o meno in questa direzione.

@Fupete
Copy link
Contributor

Fupete commented Aug 9, 2024

Esporre le didascalie in modo accessibile è un obbligo, non possiamo scegliere di non farlo.
Lascerei decidere ad @astagi come rendere meno invasivo possibile il breaking change e/o come documentarlo meglio in documentazione cc @cfabry @zetareticoli .

@zetareticoli
Copy link
Member

Dobbiamo quindi procedere con revisione del template con overlay.

@zetareticoli zetareticoli removed this from the M1-24 milestone Aug 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a11y Issue e PR riguardanti accessibilità e dintorni
Projects
Status: 📋 Backlog
Development

Successfully merging a pull request may close this issue.

3 participants